Skip to content

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

dotsdl added 3 commits March 8, 2022 21:47
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.

2 participants