Skip to content

fix subgraph adjust_graph_style#479

Merged
jchanvfx merged 1 commit intojchanvfx:mainfrom
jonassorgenfrei:fix-copy-pasting-group-nodes
Oct 2, 2025
Merged

fix subgraph adjust_graph_style#479
jchanvfx merged 1 commit intojchanvfx:mainfrom
jonassorgenfrei:fix-copy-pasting-group-nodes

Conversation

@jonassorgenfrei
Copy link
Copy Markdown
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