-
Notifications
You must be signed in to change notification settings - Fork 26
Develop ~~> Master #584
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
Develop ~~> Master #584
Conversation
… Volume instance.
Update authors for this release
Force finufft signals to be complex
…ctor_Reddy_20220207
* Refactor ImageSource <-> RelionSource, and change Simulation accordingly to populate its metadata * typo Co-authored-by: Christopher Langfield <[email protected]>
…or_Reddy Align2D refactor/updates
* take set of mrc sizes (this was the original intention) * add custom number-parsable method * accomodate floats in coordinate files * add tests covering floats in coord files
Deprecate ensure
amitsinger
left a comment
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 great!
Codecov Report
@@ Coverage Diff @@
## master #584 +/- ##
==========================================
- Coverage 87.15% 86.48% -0.68%
==========================================
Files 106 108 +2
Lines 7608 8274 +666
==========================================
+ Hits 6631 7156 +525
- Misses 977 1118 +141
Continue to review full report at Codecov.
|
|
Addition of Everything else looks good! |
janden
left a comment
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 good. Just one thing:
* Adds`ImageSource` subclasses supporting whole-micrograph/particle-coordinate sources
Missing space after “Adds”.
Merge pull request #584 from ComputationalCryoEM/develop
Draft Release Notes:
(edited with Chris's suggestion)