Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7336

Rotation of image display bug

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Firefly
    • Labels:
      None

      Description

      The rotation of a z-scale log stretched or north-up image operation fails and changes the stretch. The image becomes black/white.
      That doesn't happen with linear stretch btw.
      It looks like a server -side problem.

      Step to reproduce:
      Either by searching from Spitzer GOALS images or just uploading the attached FITS file, then stretch on 'z-scale log', then rotate (45deg). The stretch changes. Same for north-up operation.

        Attachments

          Activity

          Hide
          zhang Lijun Zhang [X] (Inactive) added a comment -

          To verify if the changes in Geom could introduce problems., I tested how the histogram and stretch handled NaN. Here is what I found:

          1. NaN is eliminated in histogram calculation
          2. The pixel value is assigned to 255 when the corresponding flux value is NaN.
          Show
          zhang Lijun Zhang [X] (Inactive) added a comment - To verify if the changes in Geom could introduce problems., I tested how the histogram and stretch handled NaN. Here is what I found: NaN is eliminated in histogram calculation The pixel value is assigned to 255 when the corresponding flux value is NaN.
          Hide
          ymei Yi Mei [X] (Inactive) added a comment -

          My previous tests showed that the rotation bug was fixed so that the image size won't be too large for the server to handle, the zscale might still have some problem. See the test case:

          When using the attached fits file, select Zscale Linear Stretch, rotate the image by 5 deg for several times, the color of the image changes after each rotation.

          I have talked with Lijun about it. I will continue to debug the zscale code.

          Show
          ymei Yi Mei [X] (Inactive) added a comment - My previous tests showed that the rotation bug was fixed so that the image size won't be too large for the server to handle, the zscale might still have some problem. See the test case: When using the attached fits file, select Zscale Linear Stretch, rotate the image by 5 deg for several times, the color of the image changes after each rotation. I have talked with Lijun about it. I will continue to debug the zscale code.
          Hide
          ymei Yi Mei [X] (Inactive) added a comment -

          Checked Zscale.java and confirmed that the amount of NaNs added by each rotation do have effect on ZScale, low and high values. That is why the color of image changes after rotations when ZScale is used.

          DS9 is doing different thing: Rotation doesn't change the fits data and only the client side handles the rotated image.

          No further debugging is needed at this point, per Trey.

          Show
          ymei Yi Mei [X] (Inactive) added a comment - Checked Zscale.java and confirmed that the amount of NaNs added by each rotation do have effect on ZScale, low and high values. That is why the color of image changes after rotations when ZScale is used. DS9 is doing different thing: Rotation doesn't change the fits data and only the client side handles the rotated image. No further debugging is needed at this point, per Trey.
          Hide
          zhang Lijun Zhang [X] (Inactive) added a comment -

          Trey approved merge of this pull request.

          Show
          zhang Lijun Zhang [X] (Inactive) added a comment - Trey approved merge of this pull request.
          Hide
          zhang Lijun Zhang [X] (Inactive) added a comment -

          Merged to dev.

          Show
          zhang Lijun Zhang [X] (Inactive) added a comment - Merged to dev.

            People

            Assignee:
            zhang Lijun Zhang [X] (Inactive)
            Reporter:
            ejoliet Emmanuel Joliet
            Reviewers:
            Emmanuel Joliet
            Watchers:
            Emmanuel Joliet, Lijun Zhang [X] (Inactive), Trey Roby, Xiuqin Wu [X] (Inactive), Yi Mei [X] (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.