-
Notifications
You must be signed in to change notification settings - Fork 91
Fix install test and uncap kaleido #4423
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
Changes from all commits
3505b67
ddcc0f2
381bd2d
9975d03
1ee459d
ce422b5
b00d604
867090c
ac13851
cf097ee
8f3af8c
8b6288f
314460c
d51cac3
54d3753
9e00b9e
6c23f1c
787a2a5
4213a84
1c7276d
5f8a18a
bceff5b
f5b99ca
34996d5
a002301
ebb35cd
dd39558
1d9201b
c65fa71
cd3f4f8
2b11a69
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,9 +16,6 @@ jobs: | |
| matrix: | ||
| os: [ubuntu-latest, macos-latest] | ||
| python_version: ["3.9", "3.10", "3.11"] | ||
| exclude: | ||
| - os: macos-latest | ||
| python-version: "3.8" | ||
| runs-on: ${{ matrix.os }} | ||
| steps: | ||
| - name: Checkout repository | ||
|
|
@@ -37,16 +34,16 @@ jobs: | |
| with: | ||
| path: ${{ env.pythonLocation }} | ||
| key: ${{ matrix.os- }}-${{ matrix.python_version }}-install-${{ env.pythonLocation }}-${{ hashFiles('**/pyproject.toml') }}-v01 | ||
| - name: Set up Homebrew | ||
| id: set-up-homebrew | ||
| uses: Homebrew/actions/setup-homebrew@master | ||
| - name: Set up cmake and libomp | ||
| run: brew install cmake libomp | ||
| - name: Build evalml package | ||
| run: make package | ||
| - name: Install evalml from sdist (not using cache) | ||
| if: steps.cache.outputs.cache-hit != 'true' | ||
| - name: Install evalml from sdist | ||
| run: | | ||
| python -m pip install "unpacked_sdist/." | ||
| - name: Install evalml from sdist (using cache) | ||
| if: steps.cache.outputs.cache-hit == 'true' | ||
| run: | | ||
| python -m pip install "unpacked_sdist/." --no-deps | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Whenever the install_tests would use cache, the test would fail. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Interesting, do you think it's noticably slower since we're not using the cache? If so, by how much? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Interestingly the time it takes varies. The latest commit (2b11a69) has the time that ubuntu takes for this step to be 8 seconds. For MacOS it takes 40 seconds for python3.9 and 3 minutes for python3.11. Conversely, the second latest commit (cd3f4f8) has the time that ubuntu takes hover around 1 minute. MacOS hovers at around 40 seconds. I can't say for sure how much slower it is without cache because it didn't work with cache enabled. I can go test to see if I can get cache working (hopefully it doesn't erase the approvals). There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FWIW, I was having trouble with caches in other OS repos recently and ended up removing them because they were failing too often. |
||
| - name: Test by importing packages | ||
| run: | | ||
| python -c "import evalml" | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -34,13 +34,16 @@ | |
| def in_container_arm64(): | ||
| """Helper fixture to run chromium as a single process for kaleido. | ||
|
|
||
| The env var is set in the Dockerfile.arm for the purposes of local | ||
| testing in a container on a mac M1, otherwise it's a noop. | ||
| Useful as kaleido > 0.1.0 can cause plotly.write_image to hang indefinitely. | ||
| Makes tests that use plotly not hang and thus pass. | ||
| """ | ||
| if os.getenv("DOCKER_ARM", None): | ||
| import plotly.io as pio | ||
| import plotly.io as pio | ||
|
|
||
| pio.kaleido.scope.chromium_args += ("--single-process",) | ||
| pio.kaleido.scope.chromium_args += ( | ||
| "--single-process", | ||
| "--headless", | ||
| "--disable-gpu", | ||
| ) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Originally these were only meant for mac m1 tests to pass, but windows tests also needs these settings for testing, else some tests hang forever, causing workers to fail. |
||
|
|
||
|
|
||
| @patch("importlib.import_module") | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.