bugGNU Astronomy Utilities - Bugs: bug #58696, Warp with --centeroncorner --scale...

 
 

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

bug #58696: Warp with --centeroncorner --scale making wrong size

Submitter:  Marjan Akbari <mrjakbari>
Submitted:  Fri 03 Jul 2020 04:59:20 PM UTC
   
 
Category:  Warp Severity:  3 - Normal
Item Group:  Output not reasonable Status:  Fixed
Privacy:  Public Assigned to:  makhlaghi
Open/Closed:  Closed

Fri 03 Jul 2020 05:37:39 PM UTC, comment #1: 

This bug was fixed and pushed to the 'master' branch as Commit 14bba0b2.

Here is the commit message (describing the problem and the fix):


Warp: not allowing --translate and --centeroncorner together

Until now, just before finding the size of the final warped image, we would
see if there is a translation element in the matrix. If there was a
translation, then the size of the output would be adjusted to account for
the translation. This is required for normal "translations" (so the image
"moves" compared to the first pixel). However, the '--centeroncorner'
option is also a translation, but this adjustment is not desired there and
will result in incorrect output sizes.

With this commit, when the '--centeroncorner' option is called, the
adjustment is not made, and to avoid unexpected problems, it won't allow
calling the '--translate' and '--centeroncorner' options together. Since
the centering is inherently a translation, a solution is suggested in the
error message.

This bug was found by Marjan Akbari.

This fixes bug #58696.


Mohammad Akhlaghi <makhlaghi>
Group administrator
Fri 03 Jul 2020 04:59:20 PM UTC, original submission:  

While working on the first (Sufi) tutorial, I noticed that after warping (command below) the output size (524) is different from the tutorial text (523, which is correct).


astwarp --scale=1/5 --centeroncorner cat_convolved.fits


Marjan Akbari <mrjakbari>

 

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

No files currently attached

 

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 mrjakbari (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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-07-03 makhlaghi StatusNone Fixed
        Assigned toNone makhlaghi
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code