show the actual x-axis units name, e.g. Nanometers
Comments (20)
-
-
reporter That’s what we decided at one of the last meeting.
Replace:
Wavelength [um] → Micrometers
Wavelength [nm] → Nanometers
etc.
-
reporter Wait, in the current version, something has changed:
That looks better. Shall we skip the “[nm]” part? Kind of redundant.
-
reporter replace
by “Datetime”
-
reporter replace
by “Decimal year”
-
reporter replace
by “Day of year”
-
reporter All only minor things, but better to read.
-
reporter Is this in QPS? If not, I can do this. Just let me know.
-
reporter -
assigned issue to
- marked as minor
-
assigned issue to
-
reporter As discussed in bigger group, we use: <physical quantity> [<abbrev. SI unit>]
For example: Wavelength [nm]
-
One remark: the decission was made on the names in the unit combo box.
We decided that x-axis name in profile plot should remain as-is.
-
reporter Not sure why this would be a good idea? Why aren’t we using the same names in both the combo box and the plot? I’m confused. Maybe I’m missing something.
-
reporter - changed status to resolved
resolves
#720→ <<cset 2e21d6b2e598>>
-
reporter - changed status to open
combobox is too big now, we need abbr. labels
-
reporter How about those?
Band Number
Band Index
Wavelength [nm]
Wavelength [um]
Wavelength [mm]
Wavelength [m]
Date [yyyy-mm-dd]
Date [yyyy.fraction]
Date [doy] -
reporter -
assigned issue to
-
assigned issue to
-
- changed status to resolved
Merged in feature/issue_720 (pull request #29)
resolves
#720Approved-by: Benjamin Jakimow
→ <<cset bf60735f2b32>>
-
SpectralLibraryIO - refactoring ASDSpectralLibraryIO - added RasterLayerSpectralLibraryIO SpectralLibraryPlot - added SpectralProfilePlotXAxisUnitModel (addresses
#720) - change X axis unit names - added basic drag and drop (needs C++ debuging :-( )Signed-off-by: Benjamin Jakimow benjamin.jakimow@geo.hu-berlin.de benjamin.jakimow@geo.hu-berlin.de
→ <<cset 8ed7aa77577e>>
-
Looks now like:
I removed the brackets for those cases where it is clear what will be shows.
For Date Time I also changes the way how labels are displayed according to the zoom level (improvable)
-
reporter Looks good! Skipping the Date prefix makes it more readable. I have the feeling, that it would be even more readable and consistent, if we would also skip the Wavelength prefix and just use Nanometers, Micrometers, Meters, etc.
Just a comment, please take it as you will.
- Log in to comment
The thickmark labels in th screenshot seems to show wavelength in micrometers, as explained by the actual axis unit label. So what is the problem here?