search

epoc-ed / GUI

Graphical User Inteface for JungFrau integrated in JEOL TEM
0 stars 0 forks source link

Reading values from redis #62

Open erikfrojdh opened 2 weeks ago

erikfrojdh commented 2 weeks ago

Related to the bug mentioned by @KiyofumiTakaba

The path for the logfile is not updated On starting, I modified the data-directory as '.../psi/UniVie/Kiyofumi/[ProjectID]/[Date]'. The path '.../psi/UniVie/CCSA/KT/[Date]' in the error message is the previous one I used. Why this value was not updated? This could be temporarily avoided by making a dummy directory of '.../psi/UniVie/CCSA/KT/[Date]'.

I think we should always read the value from redis when we need it and not rely on the cached value in the GUI.

KiyofumiTakaba commented 2 weeks ago

Probably, this is related to the issue previously raised (https://github.com/epoc-ed/GUI/issues/39). The directory path for saving data would be defined with reading the database after start writing a temporary file (RotEDlog_test.log'). At the first measurement after launching GUI, there was information about the previous measurement in the database. Even if the filepath is modified at GUI, writing 'RotEDlog_test.log' starts before updating the database. The error which I, Kiyofumi, raised would occur because I had removed the previous directory (to keep the space in /data).

kferjaoui commented 1 week ago

Indeed, as is, you'd need to restart the GUI each time that you'd change the [self.cfg.]data_dir... The problem is only for the .log file through, right?

erikfrojdh commented 1 week ago

Like @KiyofumiTakaba I think it's related to when you read the location of the data. Probably you make the logfile before the data...

kferjaoui commented 1 week ago

PR#71