Skip to content

Conversation

vtjl10
Copy link
Contributor

@vtjl10 vtjl10 commented Jun 13, 2025

Description:
This pull request corrects several typographical errors in comments and function names throughout the codebase. Specifically:

  • Fixed the typo "proided" to "provided" in a comment in train_fabric.py.
  • Corrected "prepar" to "prepare" in a comment in test_datamodules.py.
  • Fixed the function name typo "accummulated" to "accumulated" in test_tensorboard.py.

These changes improve code readability and maintain consistency in documentation and naming. No functional code was altered.


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

@github-actions github-actions bot added fabric lightning.fabric.Fabric pl Generic label for PyTorch Lightning package labels Jun 13, 2025
Copy link

codecov bot commented Jun 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81%. Comparing base (b6442c5) to head (1f9caa1).
Report is 6 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (b6442c5) and HEAD (1f9caa1). Click for more details.

HEAD has 825 uploads less than BASE
Flag BASE (b6442c5) HEAD (1f9caa1)
cpu 213 27
python 24 3
lightning_fabric 48 2
pytest 108 3
python3.10 48 6
python3.12.7 70 9
lightning 120 16
python3.11 48 6
python3.12 23 3
pytorch2.1 24 6
pytorch_lightning 48 12
pytest-full 108 27
pytorch2.2.2 12 3
pytorch2.3 12 3
pytorch2.5 12 3
pytorch2.5.1 12 3
pytorch2.4.1 12 3
pytorch2.7 12 3
pytorch2.6 12 3
Additional details and impacted files
@@            Coverage Diff            @@
##           master   #20901     +/-   ##
=========================================
- Coverage      87%      81%     -6%     
=========================================
  Files         268      268             
  Lines       23442    23442             
=========================================
- Hits        20394    18965   -1429     
- Misses       3048     4477   +1429     

@Borda Borda merged commit bd1c84e into Lightning-AI:master Jun 16, 2025
48 checks passed
Borda pushed a commit that referenced this pull request Jun 19, 2025
* Update test_tensorboard.py

* Update train_fabric.py

* Update test_datamodules.py

(cherry picked from commit bd1c84e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fabric lightning.fabric.Fabric pl Generic label for PyTorch Lightning package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants