Skip to content
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

Turn off cxi.merge integration pickles #2494

Merged
merged 3 commits into from
Aug 23, 2023
Merged

Turn off cxi.merge integration pickles #2494

merged 3 commits into from
Aug 23, 2023

Conversation

phyy-nx
Copy link
Member

@phyy-nx phyy-nx commented Aug 21, 2023

These files, output by dials.stills_process used by the legacy program cxi.merge, aren't needed anymore since cctbx.xfel.merge uses experiment lists/reflection tables.

These files, output by dials.stills_process used by the legacy program cxi.merge, aren't needed anymore since cctbx.xfel.merge uses experiment lists/reflection tables.
@codecov
Copy link

codecov bot commented Aug 22, 2023

Codecov Report

Merging #2494 (9cf9101) into main (2c4de6b) will decrease coverage by 0.05%.
The diff coverage is n/a.

❗ Current head 9cf9101 differs from pull request most recent head 84ffa43. Consider uploading reports for the commit 84ffa43 to get more accurate results

@@            Coverage Diff             @@
##             main    #2494      +/-   ##
==========================================
- Coverage   78.81%   78.76%   -0.05%     
==========================================
  Files         608      608              
  Lines       74526    74526              
  Branches    10595    10595              
==========================================
- Hits        58736    58704      -32     
- Misses      13619    13651      +32     
  Partials     2171     2171              

@ndevenish ndevenish merged commit 484c773 into main Aug 23, 2023
18 checks passed
@ndevenish ndevenish deleted the disable_int_pickles branch August 23, 2023 08:09
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.

None yet

3 participants