Skip to content

Add @override for files in src/lightning/pytorch/loops #18966

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

Merged

Conversation

VictorPrins
Copy link
Contributor

@VictorPrins VictorPrins commented Nov 7, 2023

What does this PR do?

It adds the @override decorator for all files in src/lightning/pytorch/loops (#18695).


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

@github-actions github-actions bot added the pl Generic label for PyTorch Lightning package label Nov 7, 2023
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #18966 (b67573f) into master (c524c0b) will decrease coverage by 22%.
The diff coverage is 100%.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #18966     +/-   ##
=========================================
- Coverage      70%      48%    -22%     
=========================================
  Files         450      442      -8     
  Lines       36234    36117    -117     
=========================================
- Hits        25286    17379   -7907     
- Misses      10948    18738   +7790     

@mergify mergify bot added the ready PRs ready to be merged label Nov 8, 2023
@carmocca carmocca merged commit 2334a8a into Lightning-AI:master Nov 8, 2023
@carmocca carmocca added the community This PR is from the community label Nov 8, 2023
@VictorPrins VictorPrins deleted the feature/#18695_PyTorch_loops branch November 8, 2023 18:36
@awaelchli awaelchli added this to the 2.2 milestone Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community This PR is from the community 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.

3 participants