Skip to content

[Backport version-21.0] Fix misfit plot attribute error if missing observations or missing misfit data#13262

Merged
jonathan-eq merged 2 commits intoversion-21.0from
backport-13254-to-version-21.0
Apr 8, 2026
Merged

[Backport version-21.0] Fix misfit plot attribute error if missing observations or missing misfit data#13262
jonathan-eq merged 2 commits intoversion-21.0from
backport-13254-to-version-21.0

Conversation

@scout-team-app
Copy link
Copy Markdown

@scout-team-app scout-team-app Bot commented Apr 8, 2026

Description

Backport of #13254 to version-21.0.

This commit fixes the issue where the misfit plotter would fail with `AttributeError: 'NoneType' object has no attribute 'T'` if observations for an ensemble was missing.
Now, it will plot a message `No observations available`

(cherry picked from commit 4e8f66a)
This commit fixes the issue where the misfit plotter would fail with `TypeError: unsupported operand type(s) for -: 'NoneType' and 'NoneType` if misfit data for an ensemble was missing.
Now, it will plot a message `No misfit data available`.

(cherry picked from commit 9282252)
@jonathan-eq jonathan-eq force-pushed the backport-13254-to-version-21.0 branch from 48ce5bb to 9078cc7 Compare April 8, 2026 13:13
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.53%. Comparing base (39883dc) to head (9078cc7).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@               Coverage Diff                @@
##           version-21.0   #13262      +/-   ##
================================================
- Coverage         90.55%   90.53%   -0.02%     
================================================
  Files               454      454              
  Lines             31256    31267      +11     
================================================
+ Hits              28303    28309       +6     
- Misses             2953     2958       +5     
Flag Coverage Δ
cli-tests 37.35% <0.00%> (-0.02%) ⬇️
gui-tests 68.00% <41.66%> (+0.01%) ⬆️
performance-and-unit-tests 77.19% <100.00%> (+0.01%) ⬆️
test 46.17% <16.66%> (-0.03%) ⬇️

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

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

Copy link
Copy Markdown
Contributor

@jonathan-eq jonathan-eq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jonathan-eq jonathan-eq merged commit 10e4848 into version-21.0 Apr 8, 2026
35 of 37 checks passed
@jonathan-eq jonathan-eq deleted the backport-13254-to-version-21.0 branch April 8, 2026 13:39
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