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

Add documentation for how to deal with stores beyond MemoryStore #506

Merged
merged 1 commit into from
Dec 9, 2023

Conversation

Andrew-S-Rosen
Copy link
Member

@Andrew-S-Rosen Andrew-S-Rosen commented Dec 8, 2023

Summary

This PR addresses a comment raised in openjournals/joss-reviews#5995. We now include documentation for how to modify the Jobflow settings to use a custom data store. This was already described in the API documentation, but now a concrete example is provided in the main documentation.

One open question: There is basically the same info in a portion of the Atomate2 documentation. Do we instead just link to it to prevent duplication of content? Or is there value in having a dedicated page with Jobflow. I can see arguments for both. In any case, this PR provides an example.

Copy link

codecov bot commented Dec 8, 2023

Codecov Report

Merging #506 (8384d30) into main (3b7919a) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #506   +/-   ##
=======================================
  Coverage   99.86%   99.86%           
=======================================
  Files          20       20           
  Lines        1512     1512           
  Branches      415      415           
=======================================
  Hits         1510     1510           
  Misses          2        2           

@utf
Copy link
Member

utf commented Dec 9, 2023

Great, thank you.

@utf utf merged commit 2921c88 into materialsproject:main Dec 9, 2023
9 checks passed
@utf utf added the docs Improvements or additions to documentation label Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants