ssec / sift

A visualization application for satellite imagery
http://sift.ssec.wisc.edu/
GNU General Public License v3.0
48 stars 14 forks source link

default location doesnot exist when saving image #181

Closed djhoese closed 5 years ago

djhoese commented 5 years ago

In GitLab by @tims on Oct 4, 2017, 15:45

but there seems to be no warning and no file saved.

(this was on my windows PC)

djhoese commented 5 years ago

In GitLab by @tims on Oct 4, 2017, 16:00

changed the description

djhoese commented 5 years ago

In GitLab by @djhoese on Oct 18, 2017, 12:17

assigned to @djhoese

djhoese commented 5 years ago

In GitLab by @djhoese on Oct 30, 2017, 13:12

@tims Do you have any idea what directory it defaulted to for you? It should be trying to put it in your Desktop.

djhoese commented 5 years ago

In GitLab by @djhoese on Oct 30, 2017, 13:15

Also was this with the training laptops or your own machine? Most importantly did you use the all-in-one .exe installer or the conda installer like the training laptops use.

djhoese commented 5 years ago

In GitLab by @djhoese on Nov 17, 2017, 15:37

mentioned in commit 88ba6e8ffc8761d2a2fa5d560980c54ddaa3d354

djhoese commented 5 years ago

In GitLab by @djhoese on Nov 17, 2017, 15:56

This will be fixed in the next version. The issue was related to active directory roaming profiles which you typically see in organization computer systems (like SSEC). This means a user's settings and documents are actually stored on network storage. In the SSEC case this means your user directory is something like "\\beans\users$\\data". In SIFT I was using the environment variable "%USERPROFILE%" to get the location of the user directory which is normally "C:\Users\\". It was set to this location even on the domain accounts.

To fix this I used a non-public function in the appdirs package to get the "CSIDL_DESKTOPDIRECTORY" path from the Windows system (registry maybe?) which gives the expected path for local and roaming user profiles.

djhoese commented 5 years ago

In GitLab by @djhoese on Nov 17, 2017, 15:56

closed