Skip to content

Conversation

@johannes-spies
Copy link
Contributor

@johannes-spies johannes-spies commented Oct 8, 2025

Right now, tox -e docs runs some training steps, which can take a long time. This PR adds a switch to skip training and directly builds the docs with TRAIN=0 tox -e docs.


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

@johannes-spies johannes-spies requested a review from Luthaf October 8, 2025 12:02
@johannes-spies johannes-spies changed the title Don't always retrain when creating the documentation Don't always re-train when creating the documentation Oct 8, 2025
@Luthaf
Copy link
Member

Luthaf commented Oct 8, 2025

We should merge this together with #786 which is doing something similar for the tests. Ping @HaoZeke =)

@PicoCentauri
Copy link
Contributor

Is this still necessary?

With #854 most of the trainings gets launched inside the example and will be cached. The only one left is the bash example and this is rather quick.

@johannes-spies
Copy link
Contributor Author

If they're cached, I don't see a problem and this PR here can be closed without merging. But does training happen at least once while building the documentation?

@PicoCentauri
Copy link
Contributor

Yes, when you have no docs, sphinx gallery generates the files and while doing it will call the mtt to perform training via a subprocess

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.

4 participants