Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Test of batch slicing option and still missing functions #2619

Open
astellhorn opened this issue Sep 20, 2023 · 1 comment
Open

Test of batch slicing option and still missing functions #2619

astellhorn opened this issue Sep 20, 2023 · 1 comment
Labels
Enhancement Feature requests and/or general improvements

Comments

@astellhorn
Copy link
Contributor

astellhorn commented Sep 20, 2023

In this issue I ...

(i) test the functionality of the current batch slicer and report problems.
(ii) raise still missing wishes on this option and raise improvement ideas

Here the results:

(i) Current batch slicer problems:

  • adapting the slicer manually in the 2D plot is not taken over for all 2D plots of the batch.
  • Also clicking "apply" after manual change does not do it
  • it is not possible yet for "box sum" and "circular average" (see issue Batch fitting options #2077) - also, a "wedge sum" should be made possible
  • one needs to first open every 2D plot of what shall be batch sliced, this can get very messy for >20 plots to batch slice (see issues Batch Slicer requires plotting of all 2D data sets to use in batch (Trac #1140) #1181 and Add a "select all" option for batch slicing #1743)
  • if the result of the batch slicing (1D I(Q) or I(phi) etc.) shall be plotted for all selected graphs in one 1D plot window, one first has to first save them, second, select them in the data explorer, third plot them together
    ---> this is very unintuitive, gets messy for too many graphs in the batch, and is therefore not very practical

(ii) Still missing options and improvement ideas:

@astellhorn astellhorn added the Enhancement Feature requests and/or general improvements label Sep 20, 2023
@astellhorn
Copy link
Contributor Author

Discussed during the code camp: Before it should be decided on moving the batch slicer option, following should be done first:

Step 1 - Jeff inserts the trend-option for sample environment parameters (or any other trend a user would like to dsplay)
Step 2 - for such a trend open the possibility to batch plot - i.e., having all plots in one view with tabs for opening the other graphs of that trend (Lucas comment: dynamical insert, not having all open at the same time)
--> with batch slicing all plots of one trend, and then plotting the resulting 1D curves alltogether in one plot, or also opening the possibility to plot as funciton of trend parameter
Step 3 - Annika will change the current slicer option to
Step 4 - test if then the current batch slicer needs to be improved and/or shifted to perspectives

IMG_20230921_174841_resized_20230921_054904096

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Enhancement Feature requests and/or general improvements
Projects
None yet
Development

No branches or pull requests

1 participant