|
| From: | anonymous |
| Subject: | [Octave-patch-tracker] [patch #10021] [octave forge] (mapping) meridianfwd |
| Date: | Thu, 4 Mar 2021 08:04:13 -0500 (EST) |
| User-agent: | Mozilla/5.0 (Windows NT 10.0; rv:78.0) Gecko/20100101 Firefox/78.0 |
Follow-up Comment #6, patch #10021 (project octave):
Thank you for the test cases
Adding on L142
az = wrapToPi (az);
Takes care of the imaginary portions.
The other is a quadrant issue
Doing an asin (0) but the domain is (-90, 90)
reckon is using atan2 to avoid this issue.
So as in comment #4
There needs be more for L143-144
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/patch/?10021>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
| [Prev in Thread] | Current Thread | [Next in Thread] |