Skip to content

Commit

Permalink
MAINT: Remove pytest.mark.filterwarnings(FutureWarnings) (#166)
Browse files Browse the repository at this point in the history
* MAINT: remove warning filters

Now that MNE ICALABEL .7 is out, which has my upstream fix, we should no longer need these futurewarnings as long as the CI's install .7

* FIX: remove stale GHA workflow

- This workflow inadverdently got committed in #161 i.e. it shoudl never have entered the codebase on main.
  • Loading branch information
scott-huberty authored Jul 26, 2024
1 parent 34ad17b commit ed5611b
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 68 deletions.
63 changes: 0 additions & 63 deletions .github/workflows/main_pylossless-qc-demo.yml

This file was deleted.

3 changes: 0 additions & 3 deletions pylossless/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,6 @@
import pytest


# XXX: This is a temporary fix to suppress a warning from MNE-ICAlabel
# This can be removed once MNE-ICAlabel 0.7 is released
@pytest.mark.filterwarnings("ignore::FutureWarning")
@pytest.fixture(scope="session")
def pipeline_fixture():
"""Return a namedTuple containing MNE eyetracking raw data and events."""
Expand Down
2 changes: 0 additions & 2 deletions pylossless/tests/test_rejection.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@
import pylossless as ll


# XXX: This filter can be removed once MNE-ICAlabel 0.7 is released
@pytest.mark.filterwarnings("ignore::FutureWarning")
@pytest.mark.parametrize("clean_ch_mode", [None, "drop", "interpolate"])
def test_rejection_policy(clean_ch_mode, pipeline_fixture):
"""Test the rejection policy."""
Expand Down

0 comments on commit ed5611b

Please sign in to comment.