177 views
--- tags: SPICE, Qt, Quicklook --- # 2023-07-07 Quicklook Qt meeting Participants: **David, Frédéric, Éric, Claude, Susanna** * [Previous notes](https://codimd.math.cnrs.fr/FsL3R3LRRq6sX2doKEYasQ?both#) * [Next notes](https://codimd.math.cnrs.fr/pnaSH3RzQaufDrbjpBcQQw?both) ## To do - Some icons look dark underwindows. Here is a list of standard ST icons under windows ![](https://codimd.math.cnrs.fr/uploads/upload_60ee9c9034c360b27e07ba72802dac30.png) - When loading remote data, several seconds delay before progress bar is displayed -> show progress bar before? + hourglass? - Put "Plots", "Levels" and "Zoom ~~options~~" options in (QGroups?) so that there is a frame (to isolate them better). - Rename "Zoom options" to "Zoom" - Limit "de-zoom" to original value - By default, set "lambda scaling" to 15 (not 1) - To test Gaussian fits in a difficult case: ![](https://codimd.math.cnrs.fr/uploads/upload_4b6a80b17578e184ea654c400b1fc71d.png) - Fits for this file work well (all windows) ![](https://codimd.math.cnrs.fr/uploads/upload_33a57325c76ef299bf7c5b8a0e729faf.png) - Max % textBox too small - Spectral profile bars seem to be shifted compared to fit (and then to data), bars should be centered on data points. The fitted curve (in red) could be drawn with a better x-resolution than the data ![](https://codimd.math.cnrs.fr/uploads/upload_5cc0c250ceb12a65fd88067738596c53.png) - Changing scale to logarithmic or other (and pressing Apply) has no visible effect - Use unicode λ and σ where possible - Check the correspondence of the spatial pixel vs. spectral profile. ![](https://codimd.math.cnrs.fr/uploads/upload_a25b08b3184d4e1196a67112b9baa1f1.png) ## Ready to be checked :checkered_flag: - Set tooltips everywhere to explain what buttons and options do - Force design (background white instead of low opacity on some systems) - Restore (actually remove) NaN values instead of showing zeros in the $I(\lambda)$ histogram so that the gaussian fit doesn't take into account zero values, adapt fit maps calculation as well - OK but restore real lambda index (lost when NaN are removed) - Set 1-99 % as default levels - OK - Horizontal grey bars for fit maps generation were not visible enough - OK but default range should be wider, and there should be no lower limit at 200px - Show only once the warning about the usage of those bars - OK - Show all the header information with a scroll area in the message boxes (see "i" buttons) instead of using an extra "Detailed text" button (that displays plain ugly black text) - OK - Review titles in the left menu (Appareance --> Plots, Gradient --> Color bar, ...) - OK but remove "options" in Zoom options - Review text colors in order to have a global display more "understated" and avoid red text on black backgrounds - Review colors for $I(\lambda)$ and fit for black and white backgrounds - Move the "i" button (showing the FITS file header) from menu bar to near the data file name - OK - Hide ticks (colored triangles) in the level histograms - OK :point_right: _Pyqtgraph restore 2 ticks by default when updating maps (and when user changes map colors), we force "0 tick" each time._ - Folders venv, env, .venv and .venv must be git ignored ## Doing now - Connect export button to pyqtgraph export menu :point_right: _There is an issue with that: it works only if the right click menu was open at least once before (because it's creating useful objects for pyqtgraph to make the export menu work)_ - Adapt app. to Full Spectrum input files (i.e. STUDYTYP == Single Exposure): remove map x,y, gaussian fit and fit maps - Add a button to cancel a running thread (partially done, only the button is missing: processes are already interrupted when another data file is loaded) ## Formerly checked - By double-cliking you can aim a $(x,y)$ or $(\lambda,y)$ position in one shot instead of moving horizontal and vertical bars - OK - Show thread progresses in the footer - OK - When moving a slider, update everything in real time (i.e. other related sliders, $I(\lambda)$ plot, ...) - OK - Show mouse coordinates on mouse hover (+ $I$ value) - OK - Add a button to reset zoom (triggers "View All") - OK - Fix level option especially for $log$ and $sqrt$ methods (it crashed) and prevent user to click on "Apply" twice - OK - Automatically reset zoom (View All) on $I(\lambda)$ each time it must be updated - OK - Find a way to exclude dumbbells in fit maps generation (even better: the user can now select a small region of interest thanks to the grey bars in the $I(x,y)$ map before generating it) - Bars are not much visible - No need to redisplay the message each time - Fix: reload several data files crashed the app - Working up to now - Abort processes when user loads another data file (--> interrupted progress bar in the footer changes to red) - OK - Show more study and window details (slit, ...) - Change to show all as a list - Fix: moving the $y$ horizontal bar worked for the 1st window but didn't correctly for the other ones - OK - Add an offset $+ K$ in the gaussian fit - OK - But the NaNs seem to be considered as 0 for the fit and 1D fit plot, they should remain NaN. Also for the 2D map fit. - Review the layout of the local level option in order to put side by side a maximum of window blocs, e.g. to view all the $I(x,y)$ maps - OK - Remove the $y$ slider, the $y$ value is only handled by the horizontal bars - OK - Show number of current running threads - OK - Hide ROI and Menu buttons next to maps - OK - Clarify $x, y, \lambda, t$ next to NAXIS values - OK - Do not show NaN warnings, no reason to worry, they are in the data - OK - Sometimes by default, the right part of the $I(x,y)$ map was not visible, "View All" is triggered after loading the maps to fix it - OK - Moving hardly the 3rd dimension slider of the second map shouldn't freeze the app anymore - OK - Fix: the fit maps titles were wrong after moving the mouse over one of it - OK - Add pop-ups to explain what the user can do with fit maps - OK