Skip to content

Rohdaten und erstes Anreicherungskript - #14

Merged
rdmueller merged 17 commits into
highsource:masterfrom
mfdz:geojson
Feb 3, 2025
Merged

Rohdaten und erstes Anreicherungskript#14
rdmueller merged 17 commits into
highsource:masterfrom
mfdz:geojson

Conversation

@hbruch

@hbruch hbruch commented Feb 26, 2023

Copy link
Copy Markdown
Contributor

Dieser PR stellt Rohdaten bereit, aus denen perspektivisch verschiedene Kartenansichten generiert werden können.
Zum einen authorities.csv, eine Liste von Verkehrsverbünden und teilweise Kreisen/Städten/Verkehrsunternehmen.

Zum anderen assignments.csv (bessere Namensvorschläge willkommen), die derzeit kreisweise das Abdeckungsgebiet des Verbunds/Unternehmens/Gebietskörperschaft beschreiben. Einige Kreise sind derzeit mehreren Verbünden zugeordnet (z.B. Bautzen (14625) ZVON und VVO, in der unten angehängten Karte durch Überlagerung dunkler dargestellt). Für diese Kreise muss vermutlich die Zuständigkeit auf Gemeindeebene unterschhieden werden(?). Ferner gibt es Kreise/kreisfreie Städte, die zwar einem Verkehrsverbund angehören, jedoch mit einem eigenen Verkehrsunternehmen erfasst sind. Hier bin ich noch nicht ganz schlüssig, was die beste Verfahrensweise ist. Meine Vorstellung wäre, so feingranular zu werden, dass diese Liste die für die Fahrplan- und Haltestellendaten-Veröffentlichunug verantwortlichen Organisationen widerspiegelt.

Beide Listen sind ein erster Aufschlag und sicher noch nicht fehlerfrei. Danke für Feedback zu Korrekturbedarf.

Weiterhin enthält dieser PR ein Skript, welches die o.g. Daten mit den Kreis-Geometrien (Datenquelle BKG VG250) sowie, soweit vorhanden, Wikidata-Informationen zu den Verkehrsverbünden anreichert.

Ziel sollte m.E. sein, nur solche Informationen in diesem Repository zu pflegen, die nicht besser in Wikidata gepflegt werden kann. Womöglich ist es auch erst einmal einfacher, die Informationen hier zusammenzutragen, bis geklärt ist, ob und wie sie auch in wikidata übernommen werden kann.

Schließlich beinhaltet der PR eine neue QGIS-Projekt-Datei, welche die generierte authorities_enhanced.json einbindet und (perspektivisch in verschiedenen Stilen) visulisiert.

grafik

Comment thread merge.py
}
"""

# TODO In case https://github.com/RDFLib/sparqlwrapper/pull/225 is merged

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just cross-linking RDFLib/sparqlwrapper#225 for reference here

Comment thread requirements.txt Outdated
@@ -0,0 +1,2 @@
geopandas

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we pin these packages to their major versions, in order to make the build script more reproducible long-term?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree. A >= should be a good way to keep them working for as long as possible, without breaking (assuming the dependencies follow semantic versioning).

@derhuerst

Copy link
Copy Markdown
Collaborator

Ziel sollte m.E. sein, nur solche Informationen in diesem Repository zu pflegen, die nicht besser in Wikidata gepflegt werden kann. Womöglich ist es auch erst einmal einfacher, die Informationen hier zusammenzutragen, bis geklärt ist, ob und wie sie auch in wikidata übernommen werden kann.

Ich denke auch, dass die Daten in Wikidata deutlich besser weiterverwendbar sind.

Vielleicht können wir eine der folgenden Properties nutzen, um die Zuständigkeiten direkt in Wikidata abzubilden:

Aktuell scheinen allerdings nur sehr wenige "transit districts" (``) eine dieser properties zu nutzen.

@1Maxnet1 1Maxnet1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I gave it a first try, but couldn't load the resulting geojson into kepler.gl. Will further investigate what the issue might be. Also left some proposals for improvements

Comment thread README.md Outdated
Comment thread requirements.txt Outdated
Comment thread merge.py Outdated
Comment thread merge.py Outdated
@rdmueller

Copy link
Copy Markdown
Collaborator

just noticed this PR. Shall we merge?

@hbruch

hbruch commented Feb 3, 2025

Copy link
Copy Markdown
Contributor Author

I'll have a look into @1Maxnet1's suggestions and the GeoJSON issue

@1Maxnet1

1Maxnet1 commented Feb 3, 2025

Copy link
Copy Markdown

Will further investigate what the issue might be.

I was able to fix the issue, by converting the GeoDataFrame to EPSG:4326 in line 75:

all_districts = gpd.read_file(CACHED_VG250_PATH).to_crs(epsg=4326)

Not sure whether we want to include that in the script, but at least Kepler.gl and geojson.io where not able to handle the other format, while the converted version works out-of-the-box in Kepler.gl.

@hbruch

hbruch commented Feb 3, 2025

Copy link
Copy Markdown
Contributor Author

I fixed the WFS request. Note that you have to delete your cache to re-request the WFS features in EPSG:4326.

@1Maxnet1 1Maxnet1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me now 👍

@rdmueller
rdmueller merged commit 223cb8e into highsource:master Feb 3, 2025
@rdmueller

Copy link
Copy Markdown
Collaborator

thanx! merged...

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

Successfully merging this pull request may close these issues.

4 participants