Skip to content

Commit

Permalink
Show output of msconvert.exe on error
Browse files Browse the repository at this point in the history
  • Loading branch information
naderm committed Jan 28, 2019
1 parent e30202a commit e31c423
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 2 deletions.
6 changes: 6 additions & 0 deletions ChangeLog.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# Change Log

## 0.13.9 (2019-01-28)

Bug fixes

- Show output of msconvert.exe on error.

## 0.13.8 (2019-01-25)

Bug fixes
Expand Down
10 changes: 9 additions & 1 deletion pycamv/proteowizard.py
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,15 @@ def raw_to_mzml(raw_path, out_dir, scans=None, mz_window=None):
"-c", config_path,
]

out = subprocess.check_output(cmd)
try:
out = subprocess.check_output(
cmd,
stderr=subprocess.STDOUT,
)
except subprocess.CalledProcessError as err:
LOGGER.error("Error Running msconvert:\n{}".format(err.output))
raise

encoding = sys.stdout.encoding or "utf-8"
LOGGER.debug(out.decode(encoding))

Expand Down
2 changes: 1 addition & 1 deletion pycamv/version.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
__version__ = "0.13.8"
__version__ = "0.13.9"
"""
PyCAMVerter Version.
"""

0 comments on commit e31c423

Please sign in to comment.