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

Investigating OS Error during HTML file write #78

Open
Mulan-94 opened this issue Sep 14, 2020 · 3 comments
Open

Investigating OS Error during HTML file write #78

Mulan-94 opened this issue Sep 14, 2020 · 3 comments
Labels
gains for the gain plotters

Comments

@Mulan-94
Copy link
Collaborator

Mulan-94 commented Sep 14, 2020

This investigation led to this bokeh commit and to the package tempfile

This stack abuse post gives some further information.
Tempfile is a default python package that scans the following list of directories :

  • The directory named by the TMPDIR environment variable.
  • The directory named by the TEMP environment variable.
  • The directory named by the TMP environment variable
  • Platform-specific directories:
  • On windows, C:\TEMP, C:\TMP, \TEMP, and \TMP, in the same order.
    • On other platforms, /tmp, /var/tmp, and /usr/tmp, in the same order.
    • The current working directory.

To get the default temp files location: tempfile.gettempdir()
To set tempdir: tempfile.tempdir = "/bla/bla"

I'm going to change the temp dir location if output is HTML to see if this helps

@Mulan-94 Mulan-94 added the gains for the gain plotters label Sep 14, 2020
@o-smirnov
Copy link

Yep, I think you've found the culprit! Cross-linking to caracal-pipeline/caracal#1200

@KshitijT
Copy link
Collaborator

Yay !
So would it be a separate directory than the one set by , e.g.
export SINGULARITY_TMPDIR=/idia/projects/fornax/.singularity_tmp ?

@o-smirnov
Copy link

Yeah, SINGULARITY_TMPDIR would be respected by singularity itself, but not necessarily by the container inhabitants. I've added an issue on Stimela which would be a general defensive solution. In the meantime, ragavi should contain its own fix of course.

Mulan-94 added a commit that referenced this issue Sep 14, 2020
Mulan-94 added a commit that referenced this issue Oct 2, 2020
* Fix Correlation selection.. again

* Enforce int on numcores

* Make default n_cores int

* Change temp dir location for html plots
An attempt at #78

* Delete the --gaintype arguement
Change -kx, --k-xaxis to -x, --xaxis for xaxis selection for any table
Add -y, --yaxis as alternative options to -d, --doplot
Remove choice restrictions on doplot and xaxis

* Update docs and changelog

* Allow Kcross, Df, Xf tables
Allow any combination of ys

* Change to minor release version 0.5.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gains for the gain plotters
Projects
None yet
Development

No branches or pull requests

3 participants