Mizzou-CBMI / COSMOS2

Python Scientific Pipeline Management System
GNU General Public License v3.0
71 stars 39 forks source link

Don't use the tempfile module in drm_ge #73

Closed mdpearson closed 7 years ago

mdpearson commented 7 years ago

Turns out subprocess.check_output() is almost exactly what we need here, except that it drops stderr on the floor. I copied its implementation and added stderr support to CalledProcessError to make our use case run more smoothly. No more file i/o.

If this looks good I'll apply the same fixes to drm_slurm.

egafni commented 7 years ago

Looks great to me, and a nice general function to have to boot