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

Fix docs build #19241

Merged
merged 2 commits into from Jan 8, 2024
Merged

Fix docs build #19241

merged 2 commits into from Jan 8, 2024

Conversation

awaelchli
Copy link
Member

@awaelchli awaelchli commented Jan 7, 2024

What does this PR do?

The docs build has been blocking PRs with this error:

/home/runner/work/pytorch-lightning/pytorch-lightning/docs/source-pytorch/notebooks/course_UvA-DL/07-deep-energy-based-generative-models.ipynb:871: ERROR: Content block expected for the "raw" directive; none found.

See jupyter/nbconvert#2092


📚 Documentation preview 📚: https://pytorch-lightning--19241.org.readthedocs.build/en/19241/

cc @Borda @tchaton

@github-actions github-actions bot added docs Documentation related dependencies Pull requests that update a dependency file labels Jan 7, 2024
Copy link
Contributor

github-actions bot commented Jan 7, 2024

⚡ Required checks status: All passing 🟢

Groups summary

🟢 fabric: Docs
Check ID Status
docs-make (fabric, doctest) success
docs-make (fabric, html) success

These checks are required after the changes to requirements/docs.txt.

🟢 pytorch_lightning: Docs
Check ID Status
docs-make (pytorch, doctest) success
docs-make (pytorch, html) success

These checks are required after the changes to requirements/docs.txt.

🟢 lightning_app: Docs
Check ID Status
docs-make (app, doctest) success
docs-make (app, html) success

These checks are required after the changes to requirements/docs.txt.


Thank you for your contribution! 💜

Note
This comment is automatically generated and updates for 60 minutes every 180 seconds. If you have any other questions, contact carmocca for help.

@awaelchli awaelchli added this to the 2.1.x milestone Jan 7, 2024
@awaelchli awaelchli added the priority: 0 High priority task label Jan 7, 2024
@awaelchli awaelchli changed the title WIP: Fix docs build Fix docs build Jan 7, 2024
@Borda
Copy link
Member

Borda commented Jan 7, 2024

lest just skip it I will need to fix more of them later...

@awaelchli
Copy link
Member Author

I don't understand what you are saying. Took me a while to figure out it was this dependency, so please let us unblock development.

@mergify mergify bot added the ready PRs ready to be merged label Jan 8, 2024
@awaelchli awaelchli merged commit 4d6acda into master Jan 8, 2024
37 checks passed
@awaelchli awaelchli deleted the docs/fix-build branch January 8, 2024 11:42
awaelchli added a commit that referenced this pull request Jan 30, 2024
lexierule pushed a commit that referenced this pull request Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file docs Documentation related priority: 0 High priority task ready PRs ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants