bugGNU Astronomy Utilities - Bugs: bug #50072, ImageWarp 'align' does not align...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #50072: ImageWarp 'align' does not align previously warped (rotated) images

Submitter:  Lee Kelvin <leeskelvin>
Submitted:  Tue 17 Jan 2017 10:48:48 PM UTC
   
 
Category:  Warp Severity:  4 - Important
Item Group:  Output not reasonable Status:  Fixed
Privacy:  Public Assigned to:  makhlaghi
Open/Closed:  Closed

Tue 17 Jan 2017 11:26:15 PM UTC, comment #2: 

Thank you very much Lee, for the great bug description and the attached images.

I was able to reproduce the bug and find its solution, it has been pushed to the main repo.

The fix to this bug was a simple correction, but after doing it, I noticed another fundamental bug (as fully described in the commit message) which has also been fixed with this commit.

In short, while the image would be aligned back to its original orientation, the pixel scale would be different! It was very confusing for me at first, but after digging into the code, I found the problem and was able to fix it.

The procedure to get the most recent version of Gnuastro is fully described in the Version controlled source section of the book. In short, you just have to clone it as described there, then bootstrap it. Bootstrapping just has a few extra dependencies, but once you do it, you can always stay synchronized with the latest commits.

However, if the bootstrapping is not possible for you now, I would be happy to send you the updated tarball (version 0.2.56-6184) for you to check. It should be fixed, but in the unlikely case that it isn't, please let us know on sr #109221 (on the support tracker).

Thanks again for the great bug report, it was a very interesting test, very productive report and a challenging/educative solution (for me!).

Mohammad Akhlaghi <makhlaghi>
Group administrator
Tue 17 Jan 2017 10:48:48 PM UTC, comment #1: 

This item has been reassigned from the project GNU Astronomy Utilities support tracker to your tracker.

The original report is still available at support #109221

Following are the information included in the original report:

[field #0]                  Item ID: 109221
[field #1]                 Group ID: 11425
[field #2]              Open/Closed: Open
[field #3]                 Severity: 3 - Normal
[field #4]                  Privacy: Public
[field #8]                         : Unknown support Field Display Type
[field #9]                 Category: ImageWarp
[field #10]             Submitted by: leeskelvin
[field #11]              Assigned to: None
[field #12]             Submitted on: Wed 18 Jan 2017 12:58:01 AM JST
[field #13]                  Summary: ImageWarp 'align' does not align previously warped (rotated) images
[field #14]      Original Submission: A FITS image which has been previously rotated via ImageWarp does not align back to N/E = Y/-X as expected.

The three files attached to this report help illustrate this issue. The input file 'input.fits' is a standard FITS image already aligned to N on the positive Y axis and E on the negative X axis. I initially rotate the image 45 degrees using the command:

astimgwarp --rotate=45 input.fits

This produces the output file 'input_warped.fits', attached to this report. An attempt is then made to re-align the image back to its original alignment using the command:

astimgwarp --align input_warped.fits

This produces the output file 'input_warped_warped.fits', also attached to this report. As can be seen, this output has not been re-aligned back to N/E = Y/-X. I would expect this command to re-align the image, but no operation has been performed.

Regards,
Lee Kelvin

[field #16]               Item Group: Output not reasonable
[field #17]                   Status: None
[field #18]        Component Version: None
[field #19]         Operating System: GNU/Linux
[field #20]          Reproducibility: None
[field #21]               Size (loc): None
[field #22]            Fixed Release: None
[field #23]          Planned Release: None
[field #24]                   Effort: 0.00
[field #28]                 Priority: 5 - Normal
[field #31]         Percent Complete: 0%
[field #33]                  Release: None
[field #58]     Custom Select Box #1: None
[field #59]     Custom Select Box #2: None
[field #60]     Custom Select Box #3: None
[field #61]     Custom Select Box #4: None
[field #62]     Custom Select Box #5: None
[field #63]     Custom Select Box #6: None
[field #64]     Custom Select Box #7: None
[field #65]     Custom Select Box #8: None
[field #66]     Custom Select Box #9: None
[field #67]    Custom Select Box #10: None

Mohammad Akhlaghi <makhlaghi>
Group administrator
Tue 17 Jan 2017 10:48:48 PM UTC, original submission:  

A FITS image which has been previously rotated via ImageWarp does not align back to N/E = Y/-X as expected.

The three files attached to this report help illustrate this issue. The input file 'input.fits' is a standard FITS image already aligned to N on the positive Y axis and E on the negative X axis. I initially rotate the image 45 degrees using the command:

astimgwarp --rotate=45 input.fits

This produces the output file 'input_warped.fits', attached to this report. An attempt is then made to re-align the image back to its original alignment using the command:

astimgwarp --align input_warped.fits

This produces the output file 'input_warped_warped.fits', also attached to this report. As can be seen, this output has not been re-aligned back to N/E = Y/-X. I would expect this command to re-align the image, but no operation has been performed.

Regards,
Lee Kelvin

Lee Kelvin <leeskelvin>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #39491:  input.fits added by leeskelvin (8KiB - image/fits)
file #39492:  input_warped.fits added by leeskelvin (8KiB - image/fits)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by makhlaghi (Posted a comment)
  • -email is unavailable- added by leeskelvin (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-01-17 makhlaghi CategoryNone Warp
        Severity3 - Normal 4 - Important
        Item GroupNone Output not reasonable
        StatusNone Fixed
        Assigned toNone makhlaghi
        Open/ClosedOpen Closed
    2017-01-17 makhlaghi Reassign itemGNU Astronomy Utilities, sr #109221 GNU Astronomy Utilities, bug #50072

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code