Skip to content
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

Expose trajectory protocol for optimization datasets #194

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dotsdl
Copy link
Member

@dotsdl dotsdl commented Mar 9, 2022

Description

We set the default trajectory protocol for optimizations to be
initial_and_final in order to conserve space where possible.
Choosing another option is an opt-in choice for the submitter.

Todo

  • add appropriate unit tests

Status

  • Ready to go

We set the default trajectory protocol for optimizations to be
`initial_and_final` in order to conserve space where possible.
Choosing another option is an opt-in choice for the submitter.
@dotsdl
Copy link
Member Author

dotsdl commented Mar 24, 2022

Going to wait on this for the QCFractal refactor, as there are literally two places where protocols should go on an optimization entry, and it is not clear right now which one to use. We could try just attaching to both, but this may prove to be a temporary solution anyway until the refactor is out.

We want to remove `optimization_trajectory` from QCSpec next; doesn't
belong there.
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.

1 participant