Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

uncompressed positions - leading zero in decimaldegrees <10 missing #9

Closed
ploeffler opened this issue Oct 23, 2022 · 1 comment
Closed

uncompressed positions - leading zero in decimaldegrees <10 missing #9

ploeffler opened this issue Oct 23, 2022 · 1 comment
Assignees
@rhymeswithmogul
Labels
bug Something isn't working good first issue Good for newcomers
Milestone
v1.6

Comments

@ploeffler
Copy link
Contributor

what a great tool!

but: as mentioned in the topic leading zero is missing if decimal minutes are smaller than 10

(file aprs-wx.c lines 195 and 202)

2022-10-23 15:19:24 CEST: OE6PLD-13>APRS,TCPIP*,qAC,OE6PLD-6:@231319z476.08N/01549.60E_000/0g032t072h50b10130/A=m X raspi-aprs-weather-submit/V1.5 [Invalid uncompressed location]
2022-10-23 15:21:28 CEST: OE6PLD-13>APRS,TCPIP*,qAC,OE6PLD-6:@231321z/6_@)RN"+_g032t072h50b10130/A=m X raspi-aprs-weather-submit/V1.5

regards
peter, OE6PLD
@rhymeswithmogul rhymeswithmogul self-assigned this Oct 26, 2022
@rhymeswithmogul rhymeswithmogul added bug Something isn't working good first issue Good for newcomers labels Oct 26, 2022
@rhymeswithmogul rhymeswithmogul added this to the v1.6 milestone Oct 26, 2022
rhymeswithmogul added a commit that referenced this issue Oct 26, 2022
@rhymeswithmogul
Fix bug #9 with uncompressed positions <10 min.
baea4a3 
This pads the latitude/longitude minute values with a leading zero if
it is less than ten.  Thank you to @ploeffler for discovering this bug!
@rhymeswithmogul
Copy link
Owner

Thanks for discovering that! Can you pull the Git tree and re-test? It seems to be working for me.

$ ./aprs-weather-submit -n 40 -e -75 -k KC1HBK
KC1HBK>APRS,TCPIP*:@260133z4000.00N/07500.00W_.../...t...aprs-weather-submit/1.6-git

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rhymeswithmogul rhymeswithmogul

Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
v1.6
Development

No branches or pull requests
2 participants
@ploeffler @rhymeswithmogul