Skip to content

Conversation

@simongravelle
Copy link
Member

This concerns #52

@simongravelle simongravelle self-assigned this Jan 5, 2025
@simongravelle simongravelle linked an issue Jan 5, 2025 that may be closed by this pull request
Copy link
Collaborator

@akohlmey akohlmey left a comment

Choose a reason for hiding this comment

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

OVITO should be consistently capitalized in the tutorial.

Also, both OVITO and VMD can be launched with the current system loaded from a generated data file directly from LAMMPS-GUI (Run->View in OVITO, and Run->View in VMD).

Update: It looks like I need to revise the VMD export since it doesn't correctly identify elements for tutorial 3

Co-authored-by: Axel Kohlmeyer <akohlmey@gmail.com>
@akohlmey
Copy link
Collaborator

akohlmey commented Jan 5, 2025

Update: It looks like I need to revise the VMD export since it doesn't correctly identify elements for tutorial 3

That turned out to be a bug in TopoTools.

simongravelle and others added 2 commits January 6, 2025 08:52
Co-authored-by: Axel Kohlmeyer <akohlmey@gmail.com>
@simongravelle simongravelle merged commit 6134f5c into main Jan 6, 2025
@simongravelle simongravelle deleted the added-note-on-vmd-ovito branch January 6, 2025 07:59
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.

Not a single use of dump atom

3 participants