Skip to content

feat: Detect molecules node#2459

Draft
RobBuchananCompPhys wants to merge 7 commits into
dissolve2/dynamic-outputsfrom
dissolve2/detect-molecules
Draft

feat: Detect molecules node#2459
RobBuchananCompPhys wants to merge 7 commits into
dissolve2/dynamic-outputsfrom
dissolve2/detect-molecules

Conversation

@RobBuchananCompPhys

Copy link
Copy Markdown
Contributor

WIP

@RobBuchananCompPhys RobBuchananCompPhys force-pushed the dissolve2/detect-molecules branch 2 times, most recently from 9c5237c to b20982f Compare June 1, 2026 08:57
@RobBuchananCompPhys RobBuchananCompPhys force-pushed the dissolve2/detect-molecules branch 2 times, most recently from 4a57241 to 62fc811 Compare June 5, 2026 08:50
Comment thread src/classes/fragment.h
getIndicesRecursive(atoms, indices, j->index(), exclusions);
}
}
static void getIndicesRecursive(const std::vector<std::unique_ptr<AtomClass>> &atoms, std::vector<int> &indices, int index,

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.

Make sure to add a future refactor of Fragment as an issue for Milestone 2.0

Comment thread src/classes/structure.cpp Outdated
Comment thread src/classes/structure.cpp Outdated
Comment thread src/classes/structure.h Outdated
Comment thread src/classes/structure.h Outdated
Comment thread src/nodes/detectMolecules.cpp Outdated
Comment thread src/nodes/detectMolecules.cpp Outdated
Comment thread src/nodes/detectMolecules.cpp Outdated
Comment thread src/nodes/detectMolecules.cpp Outdated
Comment thread tests/nodes/cif.cpp Outdated
@RobBuchananCompPhys RobBuchananCompPhys force-pushed the dissolve2/detect-molecules branch 2 times, most recently from 8365ae3 to d401e48 Compare June 11, 2026 13:06
@RobBuchananCompPhys RobBuchananCompPhys force-pushed the dissolve2/detect-molecules branch from a1c86e0 to 78d22c9 Compare June 17, 2026 13:25
@RobBuchananCompPhys RobBuchananCompPhys changed the base branch from develop2 to dissolve2/dynamic-outputs June 17, 2026 13:34
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