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.

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

Många kommuner har genereringsfel #43

Open
marsip opened this issue Feb 21, 2022 · 6 comments
Open

Många kommuner har genereringsfel #43

marsip opened this issue Feb 21, 2022 · 6 comments

Comments

@marsip
Copy link

marsip commented Feb 21, 2022

I dagsläget (2022-02-21) är det över 10 kommuner som har genereringsfel på download-sidan.

@atorger
Copy link
Owner

atorger commented Feb 22, 2022

NVDB har ändrat format och grejat på senaste tiden så saker har en tendens att gå sönder. Jag har tyvärr inte tid att jobba med detta nu, men återkommer vid senare tillfälle. Om det är en specifik kommun man vill jobba med som inte funkar så kanske jag kan göra en punktinsats för just den kommunen.

@atorger
Copy link
Owner

atorger commented Jun 15, 2022

Eslöv, Finspång, Leksand, Norrtälje och Salem ska nu funka igen, vilket verkar vara alla som inte funkade tidigare.

@elindberg-snapcode
Copy link

Vet inte om detta är rätt tråd, men rubriken stämmer överens i alla fall :)
Jag har försökt splitta skåne, och gå över alla kommuner, var på tre stycken går åt skogen med två olika fel:

Trelleborg

2023-05-04 10:36:49,890 - nvdb2osm - INFO - Reading file NVDB-Kollektivkorfalt.gpkg
2023-05-04 10:36:49,904 - nvdb2osm - INFO - done (8 segments)
2023-05-04 10:36:49,904 - nvdb2osm - INFO - Parsing 8 segments...
Traceback (most recent call last):
File "/home/centos/nvdb2osm/./nvdb2osm.py", line 480, in
main()
File "/home/centos/nvdb2osm/./nvdb2osm.py", line 337, in main
ways = read_nvdb_geometry(directory_or_zip, name, TAG_TRANSLATIONS[name], nvdb_total_bounds)
File "/home/centos/nvdb2osm/./nvdb2osm.py", line 107, in read_nvdb_geometry
process_tag_translations(way, tag_translations)
File "/home/centos/nvdb2osm/tag_translations.py", line 1191, in process_tag_translations
tag_translations"translator_function"
File "/home/centos/nvdb2osm/tag_translations.py", line 816, in tag_translation_Kollektivkorfalt
time_interval = parse_time_interval_tags(ti_tags)
File "/home/centos/nvdb2osm/nvdb_ti.py", line 104, in parse_time_interval_tags
output[output_key] = item"parser"
File "/home/centos/nvdb2osm/nvdb_ti.py", line 212, in parse_day_interval
return day_map[str(values[0])] + "-" + day_map[str(values[1])]
KeyError: '1.0'

Svedala & Kristianstad

2023-05-04 10:38:07,607 - nvdb2osm - INFO - Reading file NVDB-Hastighetsgrans.gpkg
2023-05-04 10:38:07,874 - nvdb2osm - INFO - done (3096 segments)
2023-05-04 10:38:07,875 - nvdb2osm - INFO - Parsing 3096 segments...
2023-05-04 10:38:08,163 - waydb - INFO - Parsing 3096 segments: 10%
..
2023-05-04 10:38:09,176 - waydb - INFO - Parsing 3096 segments: 90%
Traceback (most recent call last):
File "/home/centos/nvdb2osm/./nvdb2osm.py", line 480, in
main()
File "/home/centos/nvdb2osm/./nvdb2osm.py", line 337, in main
ways = read_nvdb_geometry(directory_or_zip, name, TAG_TRANSLATIONS[name], nvdb_total_bounds)
File "/home/centos/nvdb2osm/./nvdb2osm.py", line 107, in read_nvdb_geometry
process_tag_translations(way, tag_translations)
File "/home/centos/nvdb2osm/tag_translations.py", line 1191, in process_tag_translations
tag_translations"translator_function"
File "/home/centos/nvdb2osm/tag_translations.py", line 689, in tag_translation_Hastighetsgrans
alt_maxspeed = parse_speed_limit(tags, "HAVHAST1")
File "/home/centos/nvdb2osm/tag_translations.py", line 167, in parse_speed_limit
speed = int(speed)
ValueError: invalid literal for int() with base 10: 'varierande'

@atorger
Copy link
Owner

atorger commented May 4, 2023

Jag har pushat fixar för just de där två problem nu. Det ena berodde på att jag inte hade någon parser för hastighetsgränsen "varierande". Nu känner den igen det, men den gör ingen speciell taggning för det, för jag vet inte riktigt hur det ska taggas i så fall. Förslag mottages om någon har det. Den verkar bara användas på alternativ hastighet så det är nog inte hela världen att hoppa över de värdena.

Det andra felet är för att dagar som tidigare var "1" - "7" eller "måndag" - "söndag", numera även kan vara decimaltal, 1.0 till 7.0 tydligen :-)

@elindberg-snapcode
Copy link

Super. works like a charm :)
Verkar vara halvt omöjligt att hänga med i deras förändringar!

@atorger
Copy link
Owner

atorger commented May 5, 2023

Det är mest bara småsaker som förändrats så det brukar vara rätt lugnt, men det slutar ju funka ganska ofta och inte alltid jag märker att det hänt då jag är aktiv i kartläggning bara 1-3 månader per år. Skriptet är avsiktligt byggt så det hoppar ut med ett fel när den får nåt den inte känner igen (istället för att bara ignorera). Det hade gått att göra det mycket mer robust genom att ignorera allt den inte klarar av översätta, men då skulle risken vara stor att förändringar i datat går obemärkta förbi och man kanske börjar tappa information i översättningen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants