Skip to content
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

Feat/Add @workflow.register_node decorator #209

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Howe829
Copy link
Contributor

@Howe829 Howe829 commented Mar 15, 2024

Adding graph.register_node decorator for easily register a node to the workflow.

@hinthornw
Copy link
Contributor

oh it's kinda convenient. Will defer to @nfcampos on the approval though

@nfcampos
Copy link
Contributor

I'm not opposed to this as long as we can do it inside the existing add_node method

@Howe829
Copy link
Contributor Author

Howe829 commented Mar 16, 2024

Hi there, I've tried to make it inside the add_node method.PLTAL
It may looks a little bit tricky but this is the best way for me to make add_node as a decorator as well as thinking about the forward compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants