bugGNU Astronomy Utilities - Bugs: bug #65197, Replace confusing angular 'dim'...

 
 

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

bug #65197: Replace confusing angular 'dim' dist by angular 'diam' dist

Submitter:  Boud Roukema <boud>
Submitted:  Tue 23 Jan 2024 07:34:07 PM UTC
   
 
Category:  CosmicCalculator Severity:  3 - Normal
Item Group:  Source code Status:  Fixed
Privacy:  Public Assigned to:  boud
Open/Closed:  Closed

Fri 02 Feb 2024 03:05:35 PM UTC, comment #1: 

Thanks a lot for this nice suggestion Boud! It has been merged in Commit 8ff805a3.

About the back-compatibility, to avoid too much development overhead; the tradition has been to keep all such breaks with previous versions in a "Changed features" section of the 'NEWS' file for each release.

The 'NEWS' file is included in every release announcement and when users search the name of the old option in the 'NEWS' file, they will find the new name for it, and from which version it was changed, and a description of why that change was made.

Mohammad Akhlaghi <makhlaghi>
Group administrator
Tue 23 Jan 2024 07:34:07 PM UTC, original submission:  

The name "angulardimdist" is quite anti-intuitive, since the angular diameter distance has nothing to do with "dimensions"; but it is related  to a diameter (one way of defining it, which presumably explains the name, is the Euclidean static space distance that an extragalactic object would be at if it's angular diameter in radians were to be used to infer the object's physical (proper) size in the tangential direction).

I've proposed a commit at d6b4823f [1], which works for me, to correct from 'dim' to 'diam' in the four places I found where it occurs.

[1] https://codeberg.org/boud/gnuastro/commit/d6b4823fb46ba76146e60db6bfbc38859324dfad

Boud Roukema <boud>
Group Member

 

(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 boud (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
    2024-02-02 makhlaghi StatusNone Fixed
        Assigned toNone boud
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code