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

ci: adding testing with mac M1 #2385

Merged
merged 8 commits into from
Feb 28, 2024
Merged

ci: adding testing with mac M1 #2385

merged 8 commits into from
Feb 28, 2024

Conversation

Borda
Copy link
Member

@Borda Borda commented Feb 15, 2024

What does this PR do?

reproducing #2383
ref: https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/

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--2385.org.readthedocs.build/en/2385/

@Borda Borda added the test / CI testing or CI label Feb 15, 2024
@Borda
Copy link
Member Author

Borda commented Feb 15, 2024

@justusschock could you pls help here to check why the metrics are off?

Copy link

codecov bot commented Feb 15, 2024

Codecov Report

Merging #2385 (b34f3e8) into master (a8c11b4) will decrease coverage by 29%.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #2385     +/-   ##
========================================
- Coverage      69%     40%    -29%     
========================================
  Files         307     307             
  Lines       17348   17348             
========================================
- Hits        11957    7009   -4948     
- Misses       5391   10339   +4948     

@Borda Borda requested a review from SkafteNicki February 15, 2024 09:08
@Borda Borda added the Priority Critical task/issue label Feb 19, 2024
@Borda Borda added this to the v1.3.x milestone Feb 19, 2024
@mergify mergify bot removed the has conflicts label Feb 27, 2024
@Borda Borda changed the title adding tests for mac M1 adding tests for mac M1 [wip] Feb 27, 2024
Expected: tensor(5114.6641)
Got: tensor(5114.6636)
@Borda
Copy link
Member Author

Borda commented Feb 28, 2024

@Borda Borda changed the title adding tests for mac M1 [wip] adding tests for mac M1 Feb 28, 2024
tests/unittests/audio/test_pesq.py Outdated Show resolved Hide resolved
tests/unittests/audio/test_pesq.py Outdated Show resolved Hide resolved
@mergify mergify bot added the ready label Feb 28, 2024
@Borda Borda changed the title adding tests for mac M1 ci: adding testing with mac M1 Feb 28, 2024
@Borda Borda merged commit 4230cfe into master Feb 28, 2024
61 checks passed
@Borda Borda deleted the ci/M1 branch February 28, 2024 12:05
Borda added a commit that referenced this pull request Mar 16, 2024
* adding tests for mac M1

* RelativeAverageSpectralError
	Expected: tensor(5114.6641)
	Got: tensor(5114.6636)

* mecab
* Apply suggestions from code review

(cherry picked from commit 4230cfe)
Borda added a commit that referenced this pull request Mar 18, 2024
* adding tests for mac M1

* RelativeAverageSpectralError
	Expected: tensor(5114.6641)
	Got: tensor(5114.6636)

* mecab
* Apply suggestions from code review

(cherry picked from commit 4230cfe)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority Critical task/issue ready test / CI testing or CI topic: Image
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants