Skip to content

Commit

Permalink
Remove use of custom env tests
Browse files Browse the repository at this point in the history
  • Loading branch information
MilesCranmer committed Feb 7, 2024
1 parent 31ecc71 commit 7aa7884
Show file tree
Hide file tree
Showing 9 changed files with 3 additions and 15 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,6 @@ jobs:
- name: "Run Torch tests"
run: coverage run --append --source=pysr --omit='*/test/*' -m pysr test torch
if: ${{ matrix.test-id == 'main' }}
- name: "Run custom env tests"
run: coverage run --append --source=pysr --omit='*/test/*' -m pysr test env
- name: "Coveralls"
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/CI_Windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,5 +62,3 @@ jobs:
run: pip install torch # (optional import)
- name: "Run Torch tests"
run: python -m pysr test torch
- name: "Run custom env tests"
run: python -m pysr test env
2 changes: 1 addition & 1 deletion .github/workflows/CI_docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,4 @@ jobs:
- name: Build docker
run: docker build --platform=${{ matrix.arch }} -t pysr .
- name: Test docker
run: docker run --platform=${{ matrix.arch }} --rm pysr /bin/bash -c 'python3 -m pysr test main && python3 -m pysr test cli && python3 -m pysr test env'
run: docker run --platform=${{ matrix.arch }} --rm pysr /bin/bash -c 'python3 -m pysr test main && python3 -m pysr test cli'
2 changes: 1 addition & 1 deletion .github/workflows/CI_docker_large_nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,4 +33,4 @@ jobs:
- name: Build docker
run: docker build --platform=${{ matrix.arch }} -t pysr --build-arg JLVERSION=${{ matrix.julia-version }} --build-arg PYVERSION=${{ matrix.python-version }} .
- name: Test docker
run: docker run --platform=${{ matrix.arch }} --rm pysr /bin/bash -c 'python3 -m pysr test main && python3 -m pysr test cli && python3 -m pysr test env'
run: docker run --platform=${{ matrix.arch }} --rm pysr /bin/bash -c 'python3 -m pysr test main && python3 -m pysr test cli'
3 changes: 0 additions & 3 deletions .github/workflows/CI_large_nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,3 @@ jobs:
run: |
python -m pysr test main
python -m pysr test cli
- name: "Run new env test"
run: python -m pysr test env
if: ${{ !(matrix.os == 'windows-latest' && matrix.python-version == '3.7') }}
2 changes: 0 additions & 2 deletions .github/workflows/CI_mac.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,5 +66,3 @@ jobs:
run: pip install torch # (optional import)
- name: "Run Torch tests"
run: python -m pysr test torch
- name: "Run custom env tests"
run: python -m pysr test env
1 change: 0 additions & 1 deletion pysr/param_groupings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@
- temp_equation_file
- tempdir
- delete_tempfiles
- julia_project
- update
- julia_kwargs
- Exporting the Results:
Expand Down
1 change: 0 additions & 1 deletion pysr/sr.py
Original file line number Diff line number Diff line change
Expand Up @@ -834,7 +834,6 @@ def __init__(
self.temp_equation_file = temp_equation_file
self.tempdir = tempdir
self.delete_tempfiles = delete_tempfiles
self.julia_project = julia_project
self.update = update
self.output_jax_format = output_jax_format
self.output_torch_format = output_torch_format
Expand Down
3 changes: 1 addition & 2 deletions pysr/test/__main__.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,5 @@
parser.add_argument(
"test",
nargs="*",
help="DEPRECATED. Use `python -m pysr test [tests...]` instead."
# help="Test to run. One or more of 'main', 'env', 'jax', 'torch', 'cli'.",
help="DEPRECATED. Use `python -m pysr test [tests...]` instead.",
)

0 comments on commit 7aa7884

Please sign in to comment.