Skip to content

Conversation

@jonassorgenfrei
Copy link
Contributor

@jonassorgenfrei jonassorgenfrei commented Oct 1, 2025

Hey @jchanvfx

me again, sorry for bothering you again.
This is just a small fix for an exception i actually introduced in #469 , appologies for that.
This exception is thrown when copying group nodes into group nodes.

Traceback (most recent call last):
  File "D:\Documents\Development\NodeGraphQt\examples\hotkeys\hotkey_functions.py", line 134, in paste_nodes
    graph.paste_nodes(adjust_graph_style=False)
  File "D:\Documents\Development\NodeGraphQt\NodeGraphQt\base\graph.py", line 2087, in paste_nodes
    nodes = self._deserialize(serial_data, relative_pos=True, adjust_graph_style=adjust_graph_style)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: SubGraph._deserialize() got an unexpected keyword argument 'adjust_graph_style'

Since the Subgraph is inheriting from NodeGraph i needed to implement this same parameter in that function too.

@jonassorgenfrei jonassorgenfrei force-pushed the fix-copy-pasting-group-nodes branch from 49952e4 to e073a12 Compare October 1, 2025 21:34
@jonassorgenfrei jonassorgenfrei marked this pull request as draft October 1, 2025 21:34
@jonassorgenfrei jonassorgenfrei force-pushed the fix-copy-pasting-group-nodes branch from e073a12 to 6542b05 Compare October 1, 2025 21:35
@jonassorgenfrei jonassorgenfrei marked this pull request as ready for review October 1, 2025 21:36
@jchanvfx jchanvfx merged commit 391b440 into jchanvfx:main Oct 2, 2025
@jonassorgenfrei jonassorgenfrei deleted the fix-copy-pasting-group-nodes branch October 3, 2025 12:55
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.

2 participants