Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

uclust.py should accept a temporary directory as an argument #55

Open
ebolyen opened this issue Dec 17, 2014 · 2 comments
Open

uclust.py should accept a temporary directory as an argument #55

ebolyen opened this issue Dec 17, 2014 · 2 comments

Comments

@ebolyen
Copy link
Member

ebolyen commented Dec 17, 2014

Instead of using from tempfile import gettempdir.
related to biocore/qiime#1319

@walterst
Copy link

walterst commented Apr 8, 2015

The functions can accept a tmp_dir parameter, they just currently aren't being passed one.
We'd have to track down each time these functions are called from QIIME and pass the temp_dir from the qiime_config when they are called rather then letting them default to the gettempdir value:
uclust_cluster_from_sorted_fasta_filepath
uclust_search_and_align_from_fasta_filepath
uclust_fasta_sort_from_filepath

@gregcaporaso
Copy link
Contributor

A work-around until we have this in place will be to set the one of the three environment variables that python looks at when trying to determine the value for tempfile.tempdir (see here). These are:

  • TMPDIR
  • TEMP
  • TMP

Note that you may need to set this variable in ~/.bashrc (not ~/.bash_profile!) if the shell that will execute these jobs is not a login shell. This is likely the case if you're using QIIME's parallel wrappers. So, you'd want to do something like this:

echo "export TMPDIR=$HOME/temp" >> ~/.bashrc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants