-
Notifications
You must be signed in to change notification settings - Fork 46
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
Label rlp #1614
Conversation
dials.rl_png and test_render_3d expect set_points as is, so move the additions to a new function.
Codecov Report
@@ Coverage Diff @@
## main #1614 +/- ##
==========================================
- Coverage 66.61% 66.59% -0.03%
==========================================
Files 614 614
Lines 68771 68803 +32
Branches 9574 9578 +4
==========================================
+ Hits 45813 45816 +3
- Misses 21027 21056 +29
Partials 1931 1931 |
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.
Tested using SACLA-MPCCD-Phase3-21528-5images.h5 after running dials.import and dials.find_spots. Behaviour looks correct and visualises already existing code (nearest neighbours), so more tests aren't needed. Definitely a helpful change. Looks like build errors are just from the master/main change.
Yes, merging |
To aid orientation and reflection matching between the reciprocal lattice and image viewers, this change introduces optional labelling for the reciprocal lattice point nearest the view centre.
![Screenshot from 2021-02-26 16-55-22](https://user-images.githubusercontent.com/11502083/109331004-34999b00-7854-11eb-8a35-8f80bddaa900.png)