Adaptive alien removal (and a few bug fixes) #32
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Before phasing on a new peak, it compares the shared-object projected diffraction to the actual measured diffraction (both with a 1-pixel gaussian blur). A difference map is defined as the absolute difference between the two blurred images divided by the blurred projection. A mask is generated from all the voxels where the difference map is greater than twice its own median. While the peak is being phased, the Fourier constraint is enforced using the measurement outside the mask and the projection inside.
…aint. The amount of feedback depends on that peak's confidence value.
Removed: - adaptive_weights (replaced by adapt_trigger) Added: - adapt_trigger: Puts the adaptive weighting in the operation/trigger format - adapt_power: Basically how harshly bad peaks are punished - adapt_alien_start: When to start removing aliens - adapt_alien_threshold: Voxels are masked where the error map exceeds this times its median Renamed: - weight_X << mp_weight_X - adapt_threshold_X << peak_threshold_X
|
Ready to merge. I updated all the documentation for config_mp. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
config_mp. I still need to go in and add the documentation for those (as well as some older things...)I think there are a few more things I need to clean up before this is ready to merge, but I wanted to put it on your radar. Should be ready in a few days.