You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The reason will be displayed to describe this comment to others. Learn more.
The signature of cedalion.sigproc.quality.sci changed. There is now the additional argument sci_thresh which is required but which was not provided by the unit test.
b1bf5ce
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.
@emiddell just saw the test for this failed but have to go home now. If you have any insights to share please let me know!
b1bf5ce
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 signature of
cedalion.sigproc.quality.sci
changed. There is now the additional argumentsci_thresh
which is required but which was not provided by the unit test.