Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: Firefly
-
Labels:
-
Story Points:2
-
Epic Link:
-
Sprint:SUIT Sprint 2018-04
-
Team:Science User Interface
Description
Do the following:
- Cubes: fixed the bugs related to moving between HiPS cubes and regular hips
- Cubes: display the correct current plane number and the total plane number when switching between HiPS cube
- Cubes: honor the data_cube_* properties and show information about the 3rd dimension
- Use: value = crval3 + (cubeIdx - crpix3 ) * cdelt3
- Make field of view and target optional and then, if they are not specified by the user, default to the HiPS properties hips_initial_ra, hips_initial_dec, and hips_initial_fov. If neither are available show 180 degrees at 0,0.
Attachments
Issue Links
- links to
Activity
Ok, i will put that in
On this ticket (which I see is marked Done) or do you want a followup ticket?
For records, code changed as suggested:
DM-13989: better control of precision based on cdelt3
|
|
|
- decimalPoints= Math.abs(cdelt3)>10 ? 0 : 1- Math.trunc(Math.log10(Math.abs(cdelt3)));
|
|
|
c65903c roby <roby@ipac.caltech.edu> on 4/4/18 at 5:11 PM
|
Confirming that I see all these fixes on irsawebdev5 right now.
For #3, the display of the 3rd coordinate, I make the following suggestion: the floating-point format for the display of the number should be chosen so that it displays as many digits to the right of the decimal point as needed to be sure that a change by one plane produces a change in the two least-significant digits. Very roughly this means that the resolution for display of a single step is always better than about 10% of the step size.
The precise algorithm could be:
I am assuming that you have an int() function that always rounds down, even for negative numbers. That is different behavior than for the usual floor() function. If you only have floor(), ping me and I'll give you the slightly more complex rule.