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

Real-time refresh #57

Open
Ai00000 opened this issue Dec 27, 2021 · 4 comments
Open

Real-time refresh #57

Ai00000 opened this issue Dec 27, 2021 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@Ai00000
Copy link

Ai00000 commented Dec 27, 2021

Graph can't update Node in real time after modifying the Node class. It needs to create a new Node.

@Ai00000 Ai00000 added the enhancement New feature or request label Dec 27, 2021
@McManning
Copy link
Owner

Hi, in what way are you modifying the node class? Renaming? Adding/removing ports? Changing execution behaviour? etc.

Thanks!
Chase

@Ai00000
Copy link
Author

Ai00000 commented Dec 27, 2021

When you modify the node structure, for example, add a new port .

@JefferiesTube
Copy link

Any news on this one? While developing nodes this is a big hassle.
Renaming node classes results in errors in the console (that do not break anything, but I can't get rid of them).
Adding Ports to new nodes is not applied to already existing nodes in graphs. So any changes would require a lot of manual recreation of nodes. So a "Node Refresh" option in the context menu would be great

@McManning McManning self-assigned this Feb 2, 2022
@McManning
Copy link
Owner

I can definitely see the annoyance here. I've had to do a lot of pre-planning before making nodes myself, but I understand that workflow doesn't always work.

Unity does have a FormerlySerializedAs attribute and Bolt has it's own similar method. We'd probably have to do something like that here, or a new tool (EditorWindow) for finding/fixing graph issues.

The above is just for renaming classes/ports though - it's sort of one of the headaches of Unity's serializer/asset loader. Without hinting what something was renamed to, we can't figure out what something was previously named once assets reload. For adding ports, I would expect that to work without issue. I'll look into seeing why that isn't refreshing nodes with updates.

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

No branches or pull requests

3 participants