Skip to content

Commit

Permalink
Updated actions and pre-commit versions
Browse files Browse the repository at this point in the history
  • Loading branch information
KulikDM committed Feb 20, 2024
1 parent 668c90d commit f457468
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
steps:
- uses: actions/checkout@v4
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v4
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
Expand All @@ -41,4 +41,4 @@ jobs:
run: |
pytest -vs --doctest-modules --cov-fail-under=90 --cov-branch --cov=pythresh --cov-report term-missing --pyargs pythresh --continue-on-collection-errors
- name: Codecov
uses: codecov/codecov-action@v3
uses: codecov/codecov-action@v4
4 changes: 2 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ repos:
name: Format docstrings

- repo: https://github.com/asottile/pyupgrade
rev: v3.15.0
rev: v3.15.1
hooks:
- id: pyupgrade
args: [--py38-plus]
Expand All @@ -42,7 +42,7 @@ repos:
name: Sort imports

- repo: https://github.com/charliermarsh/ruff-pre-commit
rev: v0.1.13
rev: v0.2.2
hooks:
- id: ruff
args: [--exit-non-zero-on-fix, --fix, --line-length=180]
Expand Down
6 changes: 4 additions & 2 deletions pythresh/thresholds/decomp.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,10 @@ class DECOMP(BaseThresholder):
def __init__(self, method='PCA', random_state=1234):

self.method = method
self.method_funcs = {'NMF': NMF(random_state=random_state),
'PCA': PCA(random_state=random_state),
self.method_funcs = {'NMF': NMF(n_components=1,
random_state=random_state),
'PCA': PCA(n_components=1,
random_state=random_state),
'GRP': GaussianRandomProjection(n_components=2,
random_state=random_state),
'SRP': SparseRandomProjection(n_components=3,
Expand Down

0 comments on commit f457468

Please sign in to comment.