8000 Refactor panel positions of FormatISISSXD by toastisme · Pull Request #762 · cctbx/dxtbx · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Refactor panel positions of FormatISISSXD #762

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

Merged

Conversation

toastisme
Copy link
Contributor

This refactors panel positions of FormatISISSXD to account for differences depending on the date of data collection. Panel positions have been modified several times over the instrument's lifetime and this is now accommodated for by checking the start date of the experiment when loading the detector.

Copy link
codecov bot commented Oct 2, 2024

Codecov Report

Attention: Patch coverage is 52.63158% with 18 lines in your changes missing coverage. Please review.

Project coverage is 41.60%. Comparing base (9cbe8d5) to head (8efa2e8).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #762   +/-   ##
=======================================
  Coverage   41.59%   41.60%           
=======================================
  Files         189      189           
  Lines       17074    17103   +29     
  Branches     3276     3281    +5     
=======================================
+ Hits         7102     7115   +13     
- Misses       9318     9330   +12     
- Partials      654      658    +4     

@toastisme toastisme merged commit 2bde8b8 into cctbx:main Oct 2, 2024
14 checks passed
@toastisme toastisme deleted the refactor_formatisissxd_detector_panels branch October 2, 2024 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0