MeeusJs icon indicating copy to clipboard operation
MeeusJs copied to clipboard

Confusion with degrees and radians

Open jumpjack opened this issue 2 years ago • 0 comments

https://github.com/Fabiz/MeeusJs/blob/3f412c68efd622ada4635361fda2f783eadf2464/lib/Astro.Coord.js#L136

"Degrees" and "radians" are units of measure, hence "degrees in radians" is a nonsense, just like saying "centimeters in meters"; it should be instead "angle in radians" or "angle in degrees".

Also the function name is misleading: "dmsToDeg" should apparently return a valule in degrees, instead it returns radians; a proper name should be "dmsToDecimal" (x° x' x"" becomes x.xxxxx°).

jumpjack avatar Mar 01 '23 08:03 jumpjack