Skip to content

refactor: QML GUI stack [Part 3] -> Categorical NodeRegistry - #2607

Merged
RobBuchananCompPhys merged 3 commits into
develop2from
dissolve2/gui2-stack/part-3-node-registry-refactor-categories
Sep 22, 2026
Merged

RobBuchananCompPhys merged 3 commits into
develop2from
dissolve2/gui2-stack/part-3-node-registry-refactor-categories

Conversation

@RobBuchananCompPhys

Copy link
Copy Markdown
Contributor

No description provided.

@RobBuchananCompPhys RobBuchananCompPhys changed the title refactor: QML GUI stack [Part 3] Categorical NodeRegistry refactor: QML GUI stack [Part 3] -> Categorical NodeRegistry Sep 15, 2026
@RobBuchananCompPhys
RobBuchananCompPhys added this pull request to stack #2609 September 15, 2026 07:38
@RobBuchananCompPhys
RobBuchananCompPhys force-pushed the dissolve2/gui2-stack/part-3-node-registry-refactor-categories branch from 579c402 to 932089a Compare September 16, 2026 13:02

@trisyoungs trisyoungs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great to me. Nice catches on the various incorrect Node::type entries!

Comment thread src/nodes/registry.cpp
Comment thread src/nodes/registry.cpp Outdated
Comment thread src/nodes/registry.h Outdated

@rprospero rprospero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the new registry, though I've added a suggestion to simplify the code.

Comment thread src/gui2/models/nodeRegistryModel.cpp Outdated
Comment thread src/gui2/qml/nodeGraph/GraphView.qml
@RobBuchananCompPhys
RobBuchananCompPhys force-pushed the dissolve2/gui2-stack/part-3-node-registry-refactor-categories branch from 932089a to 7dc5f9d Compare September 21, 2026 15:00
Base automatically changed from dissolve2/gui2-stack/part-2-qml-dev-phase1 to develop2 September 22, 2026 09:20
@RobBuchananCompPhys
RobBuchananCompPhys force-pushed the dissolve2/gui2-stack/part-3-node-registry-refactor-categories branch from 8a3a45f to d8fb8d6 Compare September 22, 2026 09:20
@RobBuchananCompPhys
RobBuchananCompPhys merged commit 5266c8e into develop2 Sep 22, 2026
12 of 26 checks passed
@RobBuchananCompPhys
RobBuchananCompPhys deleted the dissolve2/gui2-stack/part-3-node-registry-refactor-categories branch September 22, 2026 10:19
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.

3 participants