-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix macos segmentation fault #3518
fix macos segmentation fault #3518
Conversation
Thanks for looking it to this. Have you been able to make any headway into determining what caused has caused the MacOS tests to suddenly start failing? It looks like the failure on the latest commit has progressed to something related to lightgbm, which seems like progress. Might I suggest restricting the macos tests to |
My commits don't feel like progress but more like guessing. I first thought that it is a new python version (since the last successful run was executed with python 3.11.7) that caused the problem on macos and there was a historical parallel in python2.x. Am now to reverting to macos11 but this shouldn't be permanent solution. Good idea with the adjustment of extras, will try it |
Here's an issue for us to discuss / debug: #3524 I've proposed that temporary hotfix over on #3525 to fix the CI so that other PRs are unblocked. However, it would be good to get to the root cause of the issue. Thanks again for taking this one on @CloseChoice ! |
Could also try the MacOS tests on |
Just put this here since it's a good reference: Here are a couple notes: |
Should probably file an issue in the torch repo, but will leave that for tomorrow |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3518 +/- ##
=======================================
Coverage 60.65% 60.65%
=======================================
Files 90 90
Lines 12722 12722
=======================================
Hits 7717 7717
Misses 5005 5005 ☔ View full report in Codecov by Sentry. |
Opened an issue on pytorch for this: pytorch/pytorch#121101 |
Great job for figuring this out! This looks good to me. There is "WIP" in the title - is this ready to merge? After this is merged, I think we should probably keep #3524 open until things are working again with the latest release of pytorch. |
Co-authored-by: connortann <71127464+connortann@users.noreply.github.com>
this is ready to merge |
Overview
Supports #3524
Description of the changes proposed in this pull request:
Checklist
[ ] Unit tests added (if fixing a bug or adding a new feature)