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

Use dials-data instead of dials_regression/refinement_test_data #2343

Merged
merged 32 commits into from
Feb 22, 2023

Conversation

dagewa
Copy link
Member

@dagewa dagewa commented Feb 17, 2023

All tests that previously used files in dials_regression/refinement_test_data now use copies of those files accessed by the public dials-data package.

@dagewa dagewa marked this pull request as ready for review February 17, 2023 10:06
I don't understand why this error occurs, because the files on
dials-data are identical copies of those on dials_regression in this
case. Nevertheless, it is reasonable to raise the tolerance, which
is still at the level of 10 nm difference in detector distance.
warning failure.

Also note that the change should be temporary and revisited later in
the year.
@codecov
Copy link

codecov bot commented Feb 20, 2023

Codecov Report

Merging #2343 (90749e4) into main (621c781) will increase coverage by 2.38%.
The diff coverage is 95.79%.

❗ Current head 90749e4 differs from pull request most recent head 4f86c82. Consider uploading reports for the commit 4f86c82 to get more accurate results

@@            Coverage Diff             @@
##             main    #2343      +/-   ##
==========================================
+ Coverage   80.72%   83.11%   +2.38%     
==========================================
  Files         589      593       +4     
  Lines       68175    68579     +404     
  Branches     9145     9210      +65     
==========================================
+ Hits        55036    57001    +1965     
+ Misses      11031     9447    -1584     
- Partials     2108     2131      +23     

@dagewa dagewa merged commit 1d5db64 into main Feb 22, 2023
@dagewa dagewa deleted the deregression-refinement-tests branch February 22, 2023 11:13
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