Skip to content

Commit

Permalink
Unpinning deps in pyproject toml (#49)
Browse files Browse the repository at this point in the history
* unpinning deps in pyproject toml

* only pinning numpy<2 till test data is regenerated
  • Loading branch information
vineetbansal authored Sep 23, 2024
1 parent deaada0 commit c4b1594
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 11 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/test_pinned_deps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,9 @@ jobs:
environment-file: environment.yml

- name: Install package
run: pip install .
# Note: editable install for the coverage step to pick up source
# correctly. (coverage run --source=src/paste3 -m pytest)
run: pip install -e .

- name: Pre-commit checks
run: pre-commit run --all-files
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/test_unpinned_deps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,8 @@ name: Test Unpinned Dependencies
on:
push:
branches: [ main ]
pull_request_target:
types: [ opened, edited]
pull_request:
branches: [ main ]
paths: ['pyproject.toml', 'requirements.txt']

jobs:
build_wheels:
Expand Down
12 changes: 6 additions & 6 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,14 @@ classifiers = [
]

dependencies = [
"anndata==0.10.8",
"scanpy==1.10.2",
"anndata",
"scanpy",
"POT",
"numpy",
"numpy<2",
"scipy",
"scikit-learn==1.5.1",
"IPython>=7.18.1",
"statsmodels==0.14.2"
"scikit-learn",
"IPython",
"statsmodels"
]
dynamic = ["version"]

Expand Down
2 changes: 1 addition & 1 deletion tests/test_paste2.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def test_partial_pairwise_align_given_cost_matrix(slices):
pd.read_csv(output_dir / "align_given_cost_matrix_pairwise_info.csv"),
rtol=1e-05,
)
assert log == expected_log
assert log == pytest.approx(expected_log)


@pytest.mark.skip
Expand Down

0 comments on commit c4b1594

Please sign in to comment.