You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
All references to /tmp should be replaced with tempfile.tempdir.
We also really need to re-do the get_tmp_filename function/method to better use of python's tempfile module. One way to do that that might be relatively painless would be to keep the interface of that function/method the same, but internally use the tempfile module. Thoughts on that? Note that cogent.app.util.get_tmp_filename is very widely used in QIIME, though it falls low in my list because it's imported in qiime.util, where some QIIME-specific defaults (e.g., temp_dir=get_qiime_temp_dir()) are set, and then imported from there.
The text was updated successfully, but these errors were encountered:
We also need to replace the teardown_module function in bipy/app/tests/test_util.py with more fine grained teardown methods in the classes that need it.
All references to
/tmp
should be replaced withtempfile.tempdir
.We also really need to re-do the
get_tmp_filename
function/method to better use of python'stempfile
module. One way to do that that might be relatively painless would be to keep the interface of that function/method the same, but internally use the tempfile module. Thoughts on that? Note thatcogent.app.util.get_tmp_filename
is very widely used in QIIME, though it falls low in my list because it's imported in qiime.util, where some QIIME-specific defaults (e.g.,temp_dir=get_qiime_temp_dir()
) are set, and then imported from there.The text was updated successfully, but these errors were encountered: