Skip to content

Commit

Permalink
skip n_threads unit test
Browse files Browse the repository at this point in the history
  • Loading branch information
Lilferrit committed Dec 6, 2024
1 parent c43c515 commit 2123894
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
7 changes: 5 additions & 2 deletions casanovo/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,12 @@ def n_workers() -> int:
int
The number of workers.
"""
# FIXME: remove multiprocessing Linux deadlock issue workaround when
# deadlock issue is resolved.
return 0

# Windows or MacOS: no multiprocessing.
# FIXME: remove multi-threading issue workaround.
if platform.system() in ["Windows", "Darwin"] or True:
if platform.system() in ["Windows", "Darwin"]:
logger.warning(
"Dataloader multiprocessing is currently not supported on Windows "
"or MacOS; using only a single thread."
Expand Down
1 change: 1 addition & 0 deletions tests/unit_tests/test_unit.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ def test_version():
assert casanovo.__version__ is not None


@pytest.mark.skip(reason="Skipping due to Linux deadlock issue")
def test_n_workers(monkeypatch):
"""Check that n_workers is correct without a GPU."""
monkeypatch.setattr("torch.cuda.is_available", lambda: False)
Expand Down

0 comments on commit 2123894

Please sign in to comment.