-
Notifications
You must be signed in to change notification settings - Fork 28
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
Release an RxNodeComponent edge on error #327
Release an RxNodeComponent edge on error #327
Conversation
…caught-exceptions
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.
Approved. But I think we should add tests to check for this on node component
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## branch-23.07 #327 +/- ##
================================================
- Coverage 73.21% 73.19% -0.02%
================================================
Files 390 390
Lines 13382 13383 +1
Branches 1009 1010 +1
================================================
- Hits 9797 9796 -1
- Misses 3585 3587 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
/merge |
/merge |
) * Allows the `MonitorStage` to be placed immediately after stages such as `MultiFileSource` which emits `fsspec.core.OpenFiles` instead of message types. * Adds more tests for `MonitorStage`, some of these test that when the stage does raise a `NotImplementedError` that the pipeline halts propperly which was ultimately fixed in MRC PRs nv-morpheus/MRC#326 and nv-morpheus/MRC#327 fixes #941 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: #942
Description
Followup to #326
Checklist