Skip to content

Commit

Permalink
Remove memray flag where not needed
Browse files Browse the repository at this point in the history
  • Loading branch information
hendrikmakait committed Nov 13, 2024
1 parent 2e5d8a8 commit c2ccc16
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/ab_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,7 @@ jobs:
DB_NAME: ${{ matrix.runtime-version }}-${{ matrix.repeat }}.db
CLUSTER_KWARGS: AB_environments/${{ matrix.runtime-version }}.cluster.yaml
H2O_DATASETS: ${{ matrix.h2o_datasets }}
MEMRAY_PROFILE: "scheduler"
run: pytest --benchmark --memray $MEMRAY_PROFILE ${{ matrix.pytest_args }}
run: pytest --benchmark ${{ matrix.pytest_args }}

- name: Dump coiled.Cluster kwargs
run: cat cluster_kwargs.merged.yaml
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -136,9 +136,8 @@ jobs:
AZURE_STORAGE_SAS_TOKEN: ${{ secrets.AZURE_STORAGE_SAS_TOKEN}}
COILED_RUNTIME_VERSION: ${{ matrix.runtime-version }}
DB_NAME: ${{ matrix.name_prefix }}-${{ matrix.os }}-py${{ matrix.python_version }}.db
MEMRAY_PROFILE: "none"
run: |
pytest --benchmark -n 4 --dist loadscope --memray ${{ env.MEMRAY_PROFILE }} ${{ env.PYTEST_MARKERS }} ${{ matrix.pytest_args }}
pytest --benchmark -n 4 --dist loadscope ${{ env.PYTEST_MARKERS }} ${{ matrix.pytest_args }}
- name: Dump coiled.Cluster kwargs
run: cat cluster_kwargs.merged.yaml || true
Expand Down

0 comments on commit c2ccc16

Please sign in to comment.