-
Notifications
You must be signed in to change notification settings - Fork 1
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
[MPI_tasks2022]: Pandora Isomemo - KernelTimeR and TimeR (~ 5 h) #8
Comments
I am checking what needs to be done still:
@isomemo Is this what you needed here? |
It takes some time (1), but the download is working: As output I got a gif (2) showing the plot for the selected steps. Seems to be fine: @isomemo Do you still need the following option?
|
@arunge the highlighted option in 1 is a slider (not an input box). You will notice below that we have input boxes for coordinates (to be used as an alternative to the buttons to move map/zoom slider). Please add there an input box to select time (for KernelTimerR and TimeR). As for the bug (point 2) this was also noticed by you previously. Can you please open the gif and check if it working? I would then do my own tests. |
The gif is working and showing (the default) 6 plots at time points 0, 1000, 2000, 3000, 4000, 5000. |
@arunge I think that this worked for you because the file was small. I tried this and hte app greyed out for KernetTimeR (same happening for TimeR) before producing the gif. I did the following:
|
New input box for the time is under review (#57). |
I can confirm this issue. The app crashed also locally for smaller time intervals. I am checking the options here. |
@isomemo With version 22.09.01 the input box is on the beta for testing. The zip option is under review. |
Feedback regarding the input box
|
@arunge just a small correction. The update to time should be done at the same time as for coordinates. However, if input boxes are blank previous values selected in the map remain the same. For instance, if I only input time but not coordinates nor zoom then there would be no change for the coordinates/zoom but only for time. Finally, please remember that this applies to both the TimeR and KernelTimeR models. |
I pushed a new version 22.09.3 (with the option to download zip files) to the beta branch that will be available after tests are completed. @isomemo Feel free to test it. 🙂 I did not check this task:
Is it already there and working? Or must it be implemented? The changes regarding
are still in progress. |
@isomemo With version 22.10.3 on the beta app I restructered the UI for zooming, lat/long (and time) in all modelling tabs. Now, one must press a button to set time, lat/long or zoom. Please, test the new UI and let me know if all works as expected. Since, this was a more complex restructering that was necessary for all modelling tabs, this is a part of the new task "Restructure (split up) scripts for modeling tabs". Regarding
Currently, there are buttons for
Which raw data should be exported additionally and what kind of resolution do you mean? |
@arunge this should be implemented for TimeR and KernelTimeR and applies to the temporal plots. I tested this for both models using LiVES data. See images below: The options seem correct! I suppose that "Time interval..." are the intervals at which the graphs is sampled? If so this is how it should work. The output file should include mean plus all options under estimation type in separate columns. However, when I press the button "Export" shown in figure a new tab is created but no give is given for download. |
You are right, I tested this only with the map, which is working, but there seems to be an issue with the time plotting function. |
I fixed the issue with version 22.12.1.
Currently the model is not recalculated to generate the series. The plot is only created dependent on the given time using the same model output. @isomemo Should we hide this option for Time course plots? Or what kind of series should be generated in that case? |
@arunge when the Time course option is selected for display indeed the time parameter (in the main window) has no application. We can leave it or hide if this takes very little programming time. |
@isomemo with version 22.12.3 on the isomemo beta app the option to create a time series in export of plots will be hidden for time course plots. |
Input box to select a specific time (as current in current input boxes for coordinate and zoom). Fix the current gif export bug in TimeR/KernelTimerR and add the option to export zip files containing the time slides as separate files (svg, png, pdf, tiff, geo-tiff). User options the same as with gif export but time slices as individual files (user can select jpg, tiff, png) saved in a zip file.
INWT Original Comments (August 21, 2021):
· export option will export raw data (not the plot) with resolution set by the user
· Gif export bug: for some reason the gif export is too resource intensive for the server, which breaks the functionality; we are in the process of identifying the reason for this issue and finding a strategy to limit resource consumption.
Update (November 2, 2021): "export bug" is most probably a timeout issue on the server. Saving as separate images in zip should be quite straightforward (necessary to add zip command utility to docker image).
The text was updated successfully, but these errors were encountered: