mtuq icon indicating copy to clipboard operation
mtuq copied to clipboard

mtuq.download_greens_tensors output path requested

Open aileen-fisher opened this issue 2 years ago • 3 comments

Reqest ability to modify GFs output path. Writing files within the mtuq install folder becomes a problem on a multi-user system where not everyone has write permissions to that path.

aileen-fisher avatar Mar 17 '23 20:03 aileen-fisher

@aileen-fisher,

Does an environment variable approach work for you? I believe the syngine download path can be specified through the environment variable SYNGINE_CACHE, but we could work on making it more robust (checking path existence, checking write permission, better documented) or implementing something else altogether. Please let us know if you have any preferences.

-Ryan

rmodrak avatar Mar 18 '23 13:03 rmodrak

Actually, perhaps a little added documentation would be helpful. SYNGINE_CACHE doesn't seem straightforward to find from just a Google search, or perhaps I'm not looking in the right places. Would you mind forwarding me your SYNGINE_CACHE reference just so that I have it?

Aileen

On 3/18/23 09:44, Ryan Modrak wrote:

@aileen-fisher https://github.com/aileen-fisher,

Does an environment variable approach work for you? I believe the syngine download path can be specified through the environment variable SYNGINE_CACHE, but we could work on making it more robust (checking path existence, checking write permission, better documented) or implementing something else altogether. Please let us know if you have any preferences.

-Ryan

— Reply to this email directly, view it on GitHub https://github.com/uafgeotools/mtuq/issues/208#issuecomment-1474855678, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6O6EFTCH6PHMZQLMNSGGR3W4W35XANCNFSM6AAAAAAV66WXUM. You are receiving this because you were mentioned.Message ID: @.***>

aileen-fisher avatar Mar 20 '23 15:03 aileen-fisher

Right now, it's just an undocumented feature implemented here:

https://github.com/rmodrak/mtuq/blob/ea5f26796d7e8f2a38db10be49190386611b98f1/mtuq/util/syngine.py#L74

Let me think about if a more robust approach is possible.

-Ryan

rmodrak avatar Mar 20 '23 22:03 rmodrak