das-developers / das2java

The original das2 library. Provides interactive publication-ready 2-D plotting
https://das2.org
GNU Lesser General Public License v3.0
4 stars 0 forks source link

requests for new horizontal interval average #10

Closed jbfaden closed 3 years ago

jbfaden commented 3 years ago

James requested that the new horizontal interval interval average have a couple more changes:

  1. Recall you added the “Horizontal Interval Average” tool that operates on a spectrogram plot. Two small issues with this: one is that the horizontal axis is blank -- would it be possible for it to pull the horizontal axis label from the parent plot, the way all the other slicers/averages do?
  2. The other issue with the “Horizontal Interval Average” is that the window title is “Horizontal Spectrogram Average”. These should presumably match, and probably the title “Horizontal Interval Average” is the better choice -- more generic -- OK?
  3. Finally, (again, when will you learn that your good deeds never go unpunished?) would you be able to add the options to the Vertical Interval Average that you recently added to the Horizontal version of the tool? The horizontal one has the choice of Average or Sum, but currently the vertical version doesn’t.
jbfaden commented 3 years ago

1 is done, that was a bit of cut-n-paste code which didn't get updated.

jbfaden commented 3 years ago

Here's data in Autoplot to test this with:

vap+inline:r=randomSeed(5334)&L=QDataSet.LABEL&T=QDataSet.TITLE&z=putProperty(1+randn(30,30),L,'ZZ')&z=putProperty(z,T,'The ZZs Be')&y=putProperty(linspace(0,1,30),L,'YY')&x=putProperty(linspace(0,1,30),L,'XX')&x,y,z
jbfaden commented 3 years ago

Vertical Spectrogram Averager -> Vertical Interval Averager as well, I think.

jbfaden commented 3 years ago
  1. is done, except you have to enlarge the GUI to see the droplist.
jbfaden commented 3 years ago

2 is done, switching both horizontal and vertical.

jbfaden commented 3 years ago

This is done.