-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Seems to be an issue with the IN7QPNE DOM file #763
Comments
IN7QPNE.dom has not been updated in 3 years so we may have missed an update. I thought I saw some traffic on the reflector about this not long ago. I will check there if someone has a new file. |
It looks like the latest file did not make it into MASTER on GitHub. It looks OK in the release. I was running it from Delphi. I still after all these years am not quite sure how we can end up with a file in the build that is not committed into MASTER. Do you know how that happens in the case of a DOM file. After a build, I would expect everything in the build to be the same as what is downloaded from MASTER. Tom |
the github of in7qpne_cty indicates the update was dropped. i will put out
a new refresh.
tks, Howie
…On Sat, May 4, 2024 at 8:12 PM Tom Schaefer ***@***.***> wrote:
I just noticed that the DOM file in the build has the NE countries
inverted. Instead of VTWNH, the DOM has WHNVT. That is incorrect. I will
check to see what the DOM checked into GitHub has.
—
Reply to this email directly, view it on GitHub
<#763>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLCUCC6XJMQNYQV7M4W7DZAV2PLAVCNFSM6AAAAABHHG7KMSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TSMRUG4YDMMI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
SIGH. Life is tough.
the file I was sent is not a lot better than the old version.
neqp county abbreviations https://neqp.org/neqp-county-abbreviations/ is
not ascii friendly to developers (as usual) -
so I will need to take some time to callup
a script that converts the insanity to 'county-abbrtev (space) county
name' in two columns.
…On Sun, May 5, 2024 at 6:38 AM Howie Hoyt ***@***.***> wrote:
the github of in7qpne_cty indicates the update was dropped. i will put out
a new refresh.
tks, Howie
On Sat, May 4, 2024 at 8:12 PM Tom Schaefer ***@***.***>
wrote:
> I just noticed that the DOM file in the build has the NE countries
> inverted. Instead of VTWNH, the DOM has WHNVT. That is incorrect. I will
> check to see what the DOM checked into GitHub has.
>
> —
> Reply to this email directly, view it on GitHub
> <#763>, or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABVLCUCC6XJMQNYQV7M4W7DZAV2PLAVCNFSM6AAAAABHHG7KMSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TSMRUG4YDMMI>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: ***@***.***>
>
|
The counties are correct in the latest download so it looks like the right DOM made it into the release. I tested and the NE counties have the state first. They are in lower case for some reason but they are still processed properly. Closing. |
I just noticed that the DOM file in the build has the NE countries inverted. Instead of VTWNH, the DOM has WHNVT. That is incorrect. I will check to see what the DOM checked into GitHub has.
The text was updated successfully, but these errors were encountered: