-
Notifications
You must be signed in to change notification settings - Fork 144
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
Updated Input REST Souce Stage to have pipeline support #1240
Conversation
/ok to test |
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. Thanks for your contribution!
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.
Approving again now that its not in draft
I'm not sure this is needed, when |
@spatil44 Looking at this again I agree with @dagardner-nv that it should already be working since the default is to use all modes. Can you double check that you cant use
|
@mdemoret-nv those commands work with Morpheus CLI, Please feel free to close this PR in that case. |
Closing this MR refer #1234 for more details |
Updated REST Source stages to support Pipeline mode.
This will close Feature Request #1234
@mdemoret-nv I have tested these changes on the SI detection NLP pipeline with REST Source and Sink stage, and it's working totally fine. Let me know if there is anything else that needs to be done to merge this PR, Happy to contribute.
I was testing SI Model workflow with REST stage using pipeline CLI, able to make it work after the above changes, the thought of contributing :)