From 9b1a4ef021f6dc2fae43edfb42edf8e4182ac253 Mon Sep 17 00:00:00 2001 From: Dan Meliza Date: Thu, 29 Aug 2024 11:52:40 -0400 Subject: [PATCH] remove windows from test runner - memmap doesn't work --- .github/workflows/tests-python.yml | 2 +- ewave.py | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/tests-python.yml b/.github/workflows/tests-python.yml index 2a69720..d91a8c0 100644 --- a/.github/workflows/tests-python.yml +++ b/.github/workflows/tests-python.yml @@ -7,7 +7,7 @@ jobs: strategy: fail-fast: false matrix: - os: [macos-13, macos-14, ubuntu-latest, windows-latest] + os: [macos-13, macos-14, ubuntu-latest] python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"] exclude: - os: macos-14 diff --git a/ewave.py b/ewave.py index 0083fbb..4c75afc 100644 --- a/ewave.py +++ b/ewave.py @@ -193,7 +193,9 @@ def read( - memmap: if False, reads the whole file into memory at once; if not, returns a numpy.memmap object using this value as the mode argument. 'c' corresponds to copy-on-write; use 'r+' to write changes to disk. Be - warned that 'w' modes may corrupt data. + warned that 'w' modes may corrupt data. Memmap may not work with + certain input types (e.g., files in zip archives) and does not currently work + on Windows. """ if self.mode == "w":