Skip to content

Conversation

awaelchli
Copy link
Contributor

@awaelchli awaelchli commented Mar 14, 2023

What does this PR do?

We are updating the course notebooks (Lightning-AI/tutorials#232) as well as the rest of the examples (Lightning-AI/tutorials#233) to Lightning 2.0. However, some examples turned out to be problematic, and we decided to temporarily exclude them until they are updated. The goal is to release 2.0 with docs that have valid tutorials.

cc @Borda

@awaelchli awaelchli added the docs Documentation related label Mar 14, 2023
@awaelchli awaelchli added this to the 2.0 milestone Mar 14, 2023
@github-actions github-actions bot added the pl Generic label for PyTorch Lightning package label Mar 14, 2023
update


Revert "update"

This reverts commit 657eccc.
@awaelchli awaelchli force-pushed the docs/exclude-examples branch from 657eccc to 3b1dd25 Compare March 14, 2023 14:37
@awaelchli awaelchli marked this pull request as ready for review March 14, 2023 17:17
@github-actions
Copy link
Contributor

github-actions bot commented Mar 14, 2023

⚡ Required checks status: All passing 🟢

Groups summary

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

These checks are required after the changes to docs/source-pytorch/conf.py, docs/source-pytorch/index.rst.


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.

@mergify mergify bot added the ready PRs ready to be merged label Mar 14, 2023
@awaelchli awaelchli requested a review from tchaton as a code owner March 15, 2023 09:47
Copy link
Member

@Borda Borda left a comment

Choose a reason for hiding this comment

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

let's wait for the commit when all updated notebooks are build
https://dev.azure.com/Lightning-AI/Tutorials/_build/results?buildId=144811&view=results

@mergify mergify bot removed the ready PRs ready to be merged label Mar 15, 2023
@mergify mergify bot added the ready PRs ready to be merged label Mar 15, 2023
@carmocca carmocca enabled auto-merge (squash) March 15, 2023 12:44
@carmocca carmocca merged commit 01834c8 into master Mar 15, 2023
@carmocca carmocca deleted the docs/exclude-examples branch March 15, 2023 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related pl Generic label for PyTorch Lightning package ready PRs ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants