Skip to content

WIP: Extend AMSResults.get_energy_landscape() with an XYZ and INP sources - #438

Open
nfaguirrec wants to merge 6 commits into
trunkfrom
NestorAguirre/PESExpReadWriteXYZ
Open

nfaguirrec wants to merge 6 commits into
trunkfrom
NestorAguirre/PESExpReadWriteXYZ

Conversation

@nfaguirrec

@nfaguirrec nfaguirrec commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Now, we can plot the energy landscapes as reaction networks (bipartite graph)

plot_reaction_network_functions_medium plot_reaction_network_functions_large

…re moved to the private class _EnergyLandscapeLayout.

SO215
2. Add reaction network plotting for energy landscapes
3. Add formula and energy coloring with adaptive node sizing and directed arrows.
4. Add small, medium, and large plotting examples with image regression tests.
SO215
2. Read preFactReact and preFactProd into the existing State attributes.
3. Use the HCNO INP fixture with energies for the medium plotting example.
3. Update fixtures, tests, and documentation
SO215
@robertrueger

Copy link
Copy Markdown
Member

This is cool stuff, but my impression is this would be more at home in the upcoming scm.results and its GUI, rather than in PLAMS.

This branch has not been deployed

No deployments
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