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

Fix (test/actions): Excluded torch==1.9.1, platform=macos-latest tests #960

Merged
merged 4 commits into from
May 28, 2024

Conversation

nickfraser
Copy link
Collaborator

Removed failing MacOSX test. See #906, #958.

@nickfraser nickfraser changed the title [DO NOT MERGE] Fix (test/actions): Excluded python=3.8, torch==1.9.1, platform=macos-latest tests [DO NOT MERGE] Fix (test/actions): Excluded torch==1.9.1, platform=macos-latest tests May 23, 2024
@nickfraser nickfraser requested a review from Giuseppe5 May 23, 2024 17:53
@nickfraser nickfraser self-assigned this May 24, 2024
@Giuseppe5
Copy link
Collaborator

Part of me still leans towards completely removing mac-os from the testing suite. We would need to go through the tests to remove any macos specific exception.

We have already several tests disabled only for MacOS and we can't debug if anything goes wrong there.

@nickfraser nickfraser changed the title [DO NOT MERGE] Fix (test/actions): Excluded torch==1.9.1, platform=macos-latest tests Fix (test/actions): Excluded torch==1.9.1, platform=macos-latest tests May 27, 2024
@nickfraser
Copy link
Collaborator Author

nickfraser commented May 28, 2024

We'll merge this for now, and revisit if/when we want to disable all MacOSX test.

@nickfraser nickfraser merged commit 7a716f7 into Xilinx:dev May 28, 2024
337 checks passed
@nickfraser nickfraser deleted the test/macosx branch May 28, 2024 10:32
Giuseppe5 pushed a commit to Giuseppe5/brevitas that referenced this pull request May 31, 2024
…tests (Xilinx#960)

* Fix (test/actions): Excluded python=3.8, torch==1.9.1, platform=macos-latest tests.

* Fix (tests/actions): Regenerated github actions YAML files.

* Fix (tests/actions): Disabled all MaxOSX, `torch==1.9.1` tests

* Fix (tests/actions): Regenerated github actions files.
nickfraser added a commit to nickfraser/brevitas that referenced this pull request Jul 22, 2024
…tests (Xilinx#960)

* Fix (test/actions): Excluded python=3.8, torch==1.9.1, platform=macos-latest tests.

* Fix (tests/actions): Regenerated github actions YAML files.

* Fix (tests/actions): Disabled all MaxOSX, `torch==1.9.1` tests

* Fix (tests/actions): Regenerated github actions files.
nickfraser added a commit that referenced this pull request Jul 22, 2024
…tests (#960)

* Fix (test/actions): Excluded python=3.8, torch==1.9.1, platform=macos-latest tests.

* Fix (tests/actions): Regenerated github actions YAML files.

* Fix (tests/actions): Disabled all MaxOSX, `torch==1.9.1` tests

* Fix (tests/actions): Regenerated github actions files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants