You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For example value 4717.11399 actually means 47 degrees and 17.11399 minutes. To convert this into degrees you have to calculate it as follows: 47 + 17.11399 / 60 which equals to ~47.2852332. You can test this on any NMEA parser and see that current library test cases are incorrect.
I fixed parsing in my fork and also removed N/S, E/W indicators by returning negative degrees when appropriate. Let me know if you want me to submit a PR like this. chemicstry@d980978
The text was updated successfully, but these errors were encountered:
I don’t know what would make most sense to you, but feel free to submit a PR, or try out the other repo, or join aerorust’s discord, it all depends on you :D
Library parses latitude/longitude degrees by just dividing float value by 100.0 https://github.com/66Origin/nmea-0183/blob/master/src/fields/distance/parsers.rs#L26 but this is incorrect.
For example value
4717.11399
actually means 47 degrees and 17.11399 minutes. To convert this into degrees you have to calculate it as follows:47 + 17.11399 / 60
which equals to~47.2852332
. You can test this on any NMEA parser and see that current library test cases are incorrect.I fixed parsing in my fork and also removed N/S, E/W indicators by returning negative degrees when appropriate. Let me know if you want me to submit a PR like this. chemicstry@d980978
The text was updated successfully, but these errors were encountered: