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

Train model for docs using GAP #236

Merged
merged 11 commits into from
Jun 7, 2024
Merged

Train model for docs using GAP #236

merged 11 commits into from
Jun 7, 2024

Conversation

frostedoyster
Copy link
Collaborator

@frostedoyster frostedoyster commented Jun 6, 2024

Fixes #235 by training a GAP model for the ASE example


📚 Documentation preview 📚: https://metatrain--236.org.readthedocs.build/en/236/

@PicoCentauri
Copy link
Contributor

In order that this works you also have to create the model as a pre_build step in at build.job inside the .readthedocs.yml. Basically copy the line of the tox.ini to the .readthedocs.yml and remove the {toxinidir}.

See

https://docs.readthedocs.io/en/stable/config-file/v2.html#build-jobs

for more details about the syntax.

tox.ini Outdated Show resolved Hide resolved
@PicoCentauri PicoCentauri force-pushed the train-docs-model branch 2 times, most recently from 0227499 to 3d7ce69 Compare June 7, 2024 12:12
@frostedoyster
Copy link
Collaborator Author

Merge

@DavideTisi DavideTisi merged commit c50a1ff into main Jun 7, 2024
13 checks passed
@DavideTisi DavideTisi deleted the train-docs-model branch June 7, 2024 13:59
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.

Remove pre-trained model for docs
3 participants