Skip to content

dont show 0.0 for parameter errors where no fitting was done - #397

Merged
rozyczko merged 1 commit into
developfrom
summary_fixes
Aug 27, 2026
Merged

dont show 0.0 for parameter errors where no fitting was done#397
rozyczko merged 1 commit into
developfrom
summary_fixes

Conversation

@rozyczko

Copy link
Copy Markdown
Member
  • Improved summary table clarity: Unfitted parameters now display an empty uncertainty cell rather than 0.0, preventing confusion between "no uncertainty" and "perfectly determined" values.

@rozyczko rozyczko added [scope] bug Bug report or fix (major.minor.PATCH) bugfix Fix to known bug [priority] high Should be prioritized soon labels Aug 27, 2026
@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.69%. Comparing base (f96fefd) to head (e1baa8c).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #397   +/-   ##
========================================
  Coverage    93.69%   93.69%           
========================================
  Files           51       51           
  Lines         4519     4519           
========================================
  Hits          4234     4234           
  Misses         285      285           
Flag Coverage Δ
integration 39.43% <0.00%> (ø)
unittests 93.69% <100.00%> (ø)

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

Files with missing lines Coverage Δ
src/easyreflectometry/summary/summary.py 90.40% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rozyczko
rozyczko merged commit ba07dde into develop Aug 27, 2026
58 of 59 checks passed
@rozyczko
rozyczko deleted the summary_fixes branch August 27, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fix to known bug [priority] high Should be prioritized soon [scope] bug Bug report or fix (major.minor.PATCH)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant