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

set MPLCONFIGDIR to tempfile.mkdtemp() cause /tmp dir to fill up #45

Open
ytidhar opened this issue Apr 28, 2019 · 2 comments
Open

set MPLCONFIGDIR to tempfile.mkdtemp() cause /tmp dir to fill up #45

ytidhar opened this issue Apr 28, 2019 · 2 comments

Comments

@ytidhar
Copy link

ytidhar commented Apr 28, 2019

the code
os.environ['MPLCONFIGDIR'] = tempfile.mkdtemp() #So that matplotlib doesnot complain stale file handle
https://github.com/rhshah/iCallSV/blob/a22782a7fadd79e030df35fa39a954021d4b29e5/iCallSV/filterAnnotatedSV.py#L30
is bad because it cause matplot lib to create new tmp dir on every import!!!

@rhshah
Copy link
Owner

rhshah commented Apr 28, 2019

Hello @ytidhar thank you for using the codebase, at the time of implementation, that was the best option, as we would create default temporary directory to be not /tmp but directory with enough space. I have not explored for better implementations, if you have suggestions, I am open to hearing them.

@ytidhar
Copy link
Author

ytidhar commented Apr 29, 2019

Hi
when we try to use this fix we discover that matplotlib is use the config dir to write some cache files as fontList.cache file and tex.cache/ dir, as this are cache files it didnt deleted so its make sense to use fix path as /home/local/tmp/[user]/.config/matplotlib/ for it

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

2 participants