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

Skip Kendall for Pytorch 2.0+ on Mac #2210

Merged
merged 6 commits into from
Nov 18, 2023
Merged

Skip Kendall for Pytorch 2.0+ on Mac #2210

merged 6 commits into from
Nov 18, 2023

Conversation

SkafteNicki
Copy link
Member

@SkafteNicki SkafteNicki commented Nov 14, 2023

What does this PR do?

Kendall test on Mac for v2.0 or higher of Pytorch are currently failing.
For now skip to get CI stable again.

Before submitting
  • Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • 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--2210.org.readthedocs.build/en/2210/

@SkafteNicki SkafteNicki added the test / CI testing or CI label Nov 14, 2023
@SkafteNicki SkafteNicki added this to the v1.2.x milestone Nov 14, 2023
Copy link

codecov bot commented Nov 14, 2023

Codecov Report

Merging #2210 (ee4d1ca) into master (beae928) will decrease coverage by 0%.
Report is 1 commits behind head on master.
The diff coverage is 67%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #2210   +/-   ##
======================================
- Coverage      87%     87%   -0%     
======================================
  Files         293     293           
  Lines       16417   16424    +7     
======================================
+ Hits        14273   14277    +4     
- Misses       2144    2147    +3     

@SkafteNicki SkafteNicki self-assigned this Nov 14, 2023
@Borda Borda enabled auto-merge (squash) November 14, 2023 17:26
@mergify mergify bot added the ready label Nov 14, 2023
@Borda Borda requested a review from a team November 15, 2023 10:12
@Borda Borda merged commit 894de4c into master Nov 18, 2023
59 checks passed
@Borda Borda deleted the skip_kendall_mac branch November 18, 2023 08:31
Borda pushed a commit that referenced this pull request Nov 30, 2023
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

(cherry picked from commit 894de4c)
Borda pushed a commit that referenced this pull request Dec 1, 2023
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

(cherry picked from commit 894de4c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready test / CI testing or CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants