-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fix cube fit toggle with spectral subset selected #3123
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3123 +/- ##
==========================================
- Coverage 88.93% 88.88% -0.06%
==========================================
Files 112 112
Lines 17411 17391 -20
==========================================
- Hits 15485 15458 -27
- Misses 1926 1933 +7 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
glad it was an easy fix! Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The diff looks straightforward to me!
This fixes a bug where toggling to Cube Fit in the model fitting plugin when a spectral subset was selected would result in a traceback and no data being available in the dataset menu. It appears that the dataset is briefly set to None in this case before the appropriate cube data is populated in the menu.