bugGNU Astronomy Utilities - Bugs: bug #62721, Radial profile script doesn't work...

 
 

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

bug #62721: Radial profile script doesn't work the angles around zero

Submitter:  Samane <samaneraji>
Submitted:  Fri 08 Jul 2022 09:08:25 AM UTC
   
 
Category:  Installed scripts Severity:  3 - Normal
Item Group:  Output not reasonable Status:  Fixed
Privacy:  Public Assigned to:  samaneraji
Open/Closed:  Closed

Thu 21 Jul 2022 10:58:02 AM UTC, comment #1: 

Thanks a lot Samane! It has been merged as the single commit Commit 5efb92b9cc.

As I was reviewing your original code, I noticed some a fix in your code that can help in readability:

  • If you look at the new 'astarithmetic' command, you will see that there is only a single call! Remember how you had used two long 'astarithmetic' commands within an 'if'? I noticed that the only difference between the two calls is the 'and' or 'or' operator: everything else was the same. So to help in readability, I simply used the 'if' statement to define a 'condition' variable and put '$condition' in the single 'astarithmetic' call.


For the other improvements, see the commit message ;-).

Mohammad Akhlaghi <makhlaghi>
Group administrator
Fri 08 Jul 2022 09:08:25 AM UTC, original submission:  

To compute the surface brightness profile with the wedge method, it needs to give the first and second angles (taking as the reference the major axis) which you need to obtain the radial profile only in between. If you want to have a radial profile along the major axis with a width of 10 degrees, the angles should be between 355 to 5 degrees. But if the first angle given to the azimuth is bigger than the second one this error appears:
"astmkcatalog: no object labels (non-zero pixels) in /home/samane/radialprof/mock/apertures.fits (hdu 1). To make a catalog, labeled regions must be defined"

Samane <samaneraji>

 

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

Attached Files
file #53404:  sample.sh added by samaneraji (705B - application/x-shellscript)

 

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 samaneraji
  • -email is unavailable- added by samaneraji
  • -email is unavailable- added by samaneraji (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 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-07-21 makhlaghi StatusIn Progress Fixed
        Open/ClosedOpen Closed
        SummaryThe first angle given to the azimuth option in the radial profile script, couldn't be larger than the second one Radial profile script doesn't work the angles around zero
    2022-07-08 samaneraji Attached File- Added sample.sh, #53404
        Carbon-Copy- Added -email is unavailable-
        Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code