New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fluctuation EM GUI #385
Fluctuation EM GUI #385
Conversation
The CI failure is not your fault, it is a change in the new version of pytest, see also pytest-dev/pytest#5412 Going to fix this in a minute... |
If you now fetch from master and push the result, CI should pass. |
Codecov Report
@@ Coverage Diff @@
## master #385 +/- ##
=========================================
+ Coverage 0.89% 0.93% +0.03%
=========================================
Files 113 109 -4
Lines 2013 1932 -81
Branches 237 225 -12
=========================================
Hits 18 18
+ Misses 1995 1914 -81
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks almost perfect; I'd like to keep all analysis classes in the SomethingAnalysis
naming scheme though. See comment inline.
Hi, to get this merged, the conflicts need to be resolved. If you pull from upstream/master, resolve conflicts, commit and push it should work. |
Thanks, merged! |
GUI for Fluctuation EM (a new type of analysis)
Calculating of SD over a ring mask