-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Example with multiple coordinate sets / animations? #5
Comments
Bump. There doesn't seem to be any way to support multiple coordinate sets in CJSON, which is a big problem for animations, dynamics, conformers, etc. |
One good way to introduce a series of coordinate sets would be to introduce some kind of iterator, including a keyword in the geometry identifying the type of coordinates, xyz, unit cell, and if this is a coordinate series (could be both xyz or unit cell in case of ab initio molecular dynamics). We will need to add a time stamp (from MD) or an iterator integer to ensure we can extract them in the correct time order.
Bert
… On Feb 12, 2017, at 6:16 PM, Geoff Hutchison ***@***.***> wroteBump. There doesn't seem to be any way to support multiple coordinate sets in CJSON, which is a big problem for animations, dynamics, conformers, etc.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Are we going to address this in CJSON or work on the new QC JSON and handle it there? |
I was going to get this in here, but got distracted by switching to a more modern JSON library. I think what we do here/in QC JSON will hopefully share a lot. Thanks for the reminder, it should not have taken this long, apologies! I think this would offer a simple way of getting smaller trajectories out of other packages in that case too. |
I have working code now, and will get the examples uploaded soon. |
This seems to already be in the wild as part of What's missing:
|
Are there examples of CJSON with multiple conformers / coordinate sets?
The text was updated successfully, but these errors were encountered: