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

Cache update docs #321

Merged
merged 3 commits into from
Feb 16, 2024
Merged

Cache update docs #321

merged 3 commits into from
Feb 16, 2024

Conversation

jthorton
Copy link
Contributor

@jthorton jthorton commented Feb 16, 2024

Description

This PR adds instructions on how to update the calculation cache with datasets in QCArchive.

Todos

Notable points that this PR has either accomplished or will accomplish.

  • Have someone test out the docs

Questions

  • Should we provide a function to grab all relevant datasets automatically from QCArchive? Do we want to have to keep this up to date?

Status

  • Ready to go

Copy link
Member

@j-wags j-wags left a comment

Choose a reason for hiding this comment

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

I've tested locally any this works brilliantly. Thanks @jthorton! I'll update releasehistory and merge.

docs/users/bespoke-executor.md Outdated Show resolved Hide resolved
docs/users/bespoke-executor.md Outdated Show resolved Hide resolved
@j-wags
Copy link
Member

j-wags commented Feb 16, 2024

Re:

Should we provide a function to grab all relevant datasets automatically from QCArchive? Do we want to have to keep this up to date?

Maybe, but this is already such a huge improvement that I don't want to delay rolling it out :-) Also, QCF is experimenting with some local caching stuff that I'm going to check out right after this. So if we're lucky we may have the option to save a local copy of the common datasets that we distribute in the bespokefit conda package(!!!)

@j-wags j-wags merged commit 36187e9 into main Feb 16, 2024
5 of 6 checks passed
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