-
Notifications
You must be signed in to change notification settings - Fork 12
Pipeline routing info #1470
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
base: main
Are you sure you want to change the base?
Pipeline routing info #1470
Conversation
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
📢 Thoughts on this report? Let us know! 🚀 New features to boost your workflow:
|
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 doesn't appear to work when running the graph. I think we will need to refactor our conditional edges to rather use the command pattern. I'll take a look at this.
I also received this error when trying to use the functions in a Python node:
apps.pipelines.exceptions.PipelineNodeRunError: 'dict' object has no attribute 'get_all_routes'
This is because the state
is actually just a dict
and not an instance of PipelineState
.
Description
resolves #1138
User Impact
User will be able to access the routing information of the pipeline
Demo
n/a
Docs and Changelog
definite yes to both