Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Visualizations now display the config value instead of type
Any
. Also, nodes affected by configuration have the suffixnode_name: config_key
.While working on this feature, I realized all high-level viz function except
display_all_functions()
failed to show config nodes (#832). This is because it's the only function to pass the config via thenodes
argument (typehamilton.node.Node
) while other functions first filter a path of nodes, which excludes the config.Instead of a large refactoring,
hamilton.graph.create_graphviz_graph()
now has a kwargconfig
which raises an exception if left toNone
. It's the caller's responsibility to have thisconfig
match the actual config value that lead to the set ofnodes
passed. This unlikely to cause problem unless people dig into internal APIs.Changes
config
kwarg tocreate_graphviz_graph()
and upstream callerscreate_graphviz_graph()
display config nodesFunctionGraph.display()
and high-levelDriver.display_
andDriver.visualize_
create_graphviz_graph()
config
kwargHow I tested this
Notes
Checklist