Skip to content

Add sample-displacement/transparency corrections and standardize the FullProf verification suite#197

Merged
AndrewSazonov merged 41 commits into
developfrom
cwl-sample-displacement-transparency
Jun 9, 2026
Merged

Add sample-displacement/transparency corrections and standardize the FullProf verification suite#197
AndrewSazonov merged 41 commits into
developfrom
cwl-sample-displacement-transparency

Conversation

@AndrewSazonov

Copy link
Copy Markdown
Member

No description provided.

AndrewSazonov and others added 30 commits June 8, 2026 18:57
Mirrors fit() as the non-fitting trigger that refreshes structures and
experiments so experiment.data.intensity_calc reflects current params.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Loaders take (project_dir, file) and hide bundled_reference_dir; add
load_fullprof_calc_profile for .prf Icalc minus real .bac background,
handling both .bac layouts (two-column and header+flat-array).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Proposes a dirty-flag + compute-once-per-change lazy recompute of the
calculated pattern so explicit calculate() becomes optional.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Zero all refinement codes (and remove VARY/BKG auto-refine keywords) so
fp2k recomputes at fixed values; regenerate .prf/.bac/.sub references.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Per-engine sections with constants read from the frozen .pcr, Load
reference after Define-structure, explicit project.analysis.calculate(),
engine-specific profiles (SyCos/SySin, TCH-FCJ) in their own sections.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Constants from frozen .pcr, per-engine sections, fit both engines for
the TOF scale convention with after-fit agreement.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
AndrewSazonov and others added 3 commits June 9, 2026 18:39
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@AndrewSazonov AndrewSazonov added the [scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH) label Jun 9, 2026
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.37%. Comparing base (2a6c0e1) to head (938e1df).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #197      +/-   ##
===========================================
- Coverage    90.66%   90.37%   -0.30%     
===========================================
  Files          265      265              
  Lines        22494    22625     +131     
  Branches      2612     2635      +23     
===========================================
+ Hits         20394    20447      +53     
- Misses        1577     1647      +70     
- Partials       523      531       +8     
Flag Coverage Δ
unittests 90.37% <ø> (-0.30%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/easydiffraction/analysis/analysis.py 93.18% <ø> (-0.27%) ⬇️
...rc/easydiffraction/analysis/calculators/crysfml.py 40.40% <ø> (+0.82%) ⬆️
src/easydiffraction/analysis/calculators/cryspy.py 33.91% <ø> (+3.66%) ⬆️
src/easydiffraction/analysis/verification.py 61.64% <ø> (-28.07%) ⬇️
...datablocks/experiment/categories/instrument/cwl.py 100.00% <ø> (ø)
src/easydiffraction/display/plotters/plotly.py 94.04% <ø> (+<0.01%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

AndrewSazonov and others added 7 commits June 9, 2026 19:32
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@AndrewSazonov AndrewSazonov merged commit 6711fba into develop Jun 9, 2026
15 of 16 checks passed
@AndrewSazonov AndrewSazonov deleted the cwl-sample-displacement-transparency branch June 9, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant