Skip to content

Add some Laue tests#761

Merged
psavery merged 2 commits intomasterfrom
laue-tests
Feb 26, 2025
Merged

Add some Laue tests#761
psavery merged 2 commits intomasterfrom
laue-tests

Conversation

@psavery
Copy link
Copy Markdown
Collaborator

@psavery psavery commented Feb 12, 2025

This tests both simulating Laue spots as well as auto-picking Laue spots from image data.

This verifies that the simulated Laue spots match what we expect.

Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
This verifies that spots for several HKLs were correctly auto-picked.

Signed-off-by: Patrick Avery <patrick.avery@kitware.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 40.38%. Comparing base (c768f7f) to head (dec700e).
Report is 24 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #761      +/-   ##
==========================================
+ Coverage   39.83%   40.38%   +0.54%     
==========================================
  Files         142      142              
  Lines       22764    22764              
==========================================
+ Hits         9068     9193     +125     
+ Misses      13696    13571     -125     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@psavery psavery requested a review from saransh13 February 12, 2025 20:42
@psavery psavery requested a review from bnmajor February 25, 2025 17:28
@psavery psavery merged commit 2e193cf into master Feb 26, 2025
7 checks passed
@psavery psavery deleted the laue-tests branch February 26, 2025 15:18
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