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 DifferenceCompositor not using metadata from YAML #2054
Conversation
comp = DifferenceCompositor(name='diff') | ||
res = comp((self.ds1, self.ds2), standard_name='temperature_difference') | ||
comp = DifferenceCompositor(name='diff', standard_name='temperature_difference') | ||
res = comp((self.ds1, self.ds2)) |
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.
This test was incorrect. Passing standard_name
to the __call__
method of a compositor will never happen. Only the Scene calls __call__
and it passed the DataID properties computed during dependency decisions. The __init__
method is when metadata from the YAML are passed.
Codecov Report
@@ Coverage Diff @@
## main #2054 +/- ##
=======================================
Coverage 93.79% 93.79%
=======================================
Files 282 282
Lines 42170 42171 +1
=======================================
+ Hits 39554 39555 +1
Misses 2616 2616
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM
Found out the
standard_name:
I was assigning in the YAML wasn't being assigned.