set matplotlib cache directory

This MR resolves issues with matplotlib lock files.

Matplotlib uses a cache directory in the user's home directory by default. For a plotting job running in condor, the user's home directory is accessed over the network, which introduces latency and causes lock file errors. The change in this MR uses the previously empty __init__.py script in gstlal.plots to set a special matplotlib environment variable, telling it to use local tmp space for its cache directory.

requested review from @chad-hanna

assigned to @cort.posnansky

added 4 commits

• 6a47cf91 - 1 commit from branch master
• 4bf51160 - gstlal/python/plots/__init__.py: set matplotlib cache directory to local tmp space
• 895e7c3e - gstlal/python/plots/__init__.py: remove warning about import order
• 4a20619d - gstlal/python/plots/__init__.py: add copyright

Compare with previous version

marked this merge request as ready

changed title from Draft: Master matplotlib lockfile to set matplotlib cache directory

changed the description

@cort.posnansky you probably ought to include the authors of gstlal.plots more broadly since this shows up in the init file, otherwise fine.

added 1 commit

• 506b38d2 - gstlal/python/plots/__init__.py: correct copyright

Compare with previous version

I think it'd be better if this was moved to a function, either in __init__.py or in util.py. Modules shouldn't have side effects when they're imported.

added 2 commits

• 2558baf9 - gstlal/python/plots/__init__.py: embed code in a function
• 05932fec - set_matplotlib_cache_directory in gstlal_inspiral_plot_background and...

Compare with previous version

added 1 commit

• a7b4d3a9 - add set_matplotlib_cache_directory() to all plotting programs

Compare with previous version

added 1 commit

• 70058567 - fix accidental reversion of copyright

Compare with previous version

This has been tested in an offline analysis using the small bank and mass model on 6 hours of GPS time, and the dag finished without error.

added 1 commit

• 2bdb0da0 - gstlal/python/plots/__init__.py: remove option to set MPL cache to TMPDIR, and add FIXME tag

Compare with previous version

resolved all threads

added 11 commits

• 2bdb0da0...90dabd83 - 2 commits from branch master
• 6cce4833 - gstlal/python/plots/__init__.py: set matplotlib cache directory to local tmp space
• 978315ef - gstlal/python/plots/__init__.py: remove warning about import order
• d1933d88 - gstlal/python/plots/__init__.py: add copyright
• c9884e94 - gstlal/python/plots/__init__.py: correct copyright
• 605de0db - gstlal/python/plots/__init__.py: embed code in a function
• 15df4522 - set_matplotlib_cache_directory in gstlal_inspiral_plot_background and...
• 523f3460 - add set_matplotlib_cache_directory() to all plotting programs
• 9ae3eade - fix accidental reversion of copyright
• d2a39f6b - gstlal/python/plots/__init__.py: remove option to set MPL cache to TMPDIR, and add FIXME tag

Compare with previous version

enabled an automatic merge when the pipeline for d2a39f6b succeeds

merged