Skip to content

Commit

Permalink
Bugfix: Create out_dir/tmp in parsebam if needed
Browse files Browse the repository at this point in the history
Temp files from PyCoverM are stored in out_dir/tmp/pycoverm, but the tmp
directory was not created by the parsebam module. This led to a failure to make
the pycoverm dir, crashing Vamb.

Fix issue #167
  • Loading branch information
jakobnissen committed Apr 28, 2023
1 parent efa3e5a commit b486b6f
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
# Changelog

## v4.1.1
* Create tmp directory in parsebam if needed for pycoverm (issue # 167)

## v4.1.0
* Fix typo in output AAE_Z cluster names. They are now called e.g. "aae_z_1"
instead of "aae_z1"
Expand Down
2 changes: 1 addition & 1 deletion vamb/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
7) Split bins using vamb.vambtools
"""

__version__ = (4, 1, 0)
__version__ = (4, 1, 1)

from . import vambtools
from . import parsebam
Expand Down
2 changes: 1 addition & 1 deletion vamb/parsebam.py
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ def chunkwise_loading(
target_refhash: Optional[bytes],
mask: _np.ndarray,
) -> A:
_os.mkdir(cache_directory)
_os.makedirs(cache_directory)

chunks = [
(i, min(len(paths), i + nthreads)) for i in range(0, len(paths), nthreads)
Expand Down

0 comments on commit b486b6f

Please sign in to comment.