torchmetrics icon indicating copy to clipboard operation
torchmetrics copied to clipboard

experimenting with error improvements

Open Borda opened this issue 1 year ago • 1 comments

What does this PR do?

taking inspiration from https://bigl.es/tooling-tuesday-pretty-errors/ and https://towardsdatascience.com/one-line-of-code-to-say-goodbye-to-confusing-python-error-messages-8090ee6dd046

Before submitting
  • [ ] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • [x] Did you read the contributor guideline, Pull Request section?
  • [ ] Did you make sure to update the docs?
  • [x] Did you write any new necessary tests?
PR review

Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃


📚 Documentation preview 📚: https://torchmetrics--2431.org.readthedocs.build/en/2431/

Borda avatar Mar 05 '24 20:03 Borda

Codecov Report

Merging #2431 (1b30064) into master (822dba2) will increase coverage by 0%. The diff coverage is 100%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #2431   +/-   ##
======================================
  Coverage      69%     69%           
======================================
  Files         307     307           
  Lines       17410   17412    +2     
======================================
+ Hits        11991   11993    +2     
  Misses       5419    5419           

codecov[bot] avatar Mar 05 '24 21:03 codecov[bot]