Skip to content

11 new patch files#30

Open
ThorstenFroehlinghaus wants to merge 1 commit intoParkenDD:mainfrom
mobidata-bw:patch_files
Open

11 new patch files#30
ThorstenFroehlinghaus wants to merge 1 commit intoParkenDD:mainfrom
mobidata-bw:patch_files

Conversation

@ThorstenFroehlinghaus
Copy link
Collaborator

No description provided.

"max_height": 200,
"fee_description": "Spieltag: PKW/Motorrad 15,00 €; Bus 25,00 €; Kleinbus 20,00 €; Wohnmobil 20,00 €; Spielfrei: PKW/Motorrad 5,00 € (45 Min. frei); Bus/Wohnmobil 20,00 €; Wohnmobil Übernachtung: 20,00 €",
"lat": "48.214808",
"lon": "11.624268",
Copy link
Collaborator

@AbdullahiFatola AbdullahiFatola Mar 13, 2026

Choose a reason for hiding this comment

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

These coordinates are not the Entrance of the parking space but a midpoint. The data source seems to have coordinates more inclined towards the entrance. Therefore, I would not override the existing coordinates from the data source.

Generally I would avoid patching coordinates different from what the data source provides. The best way would be that the data provider amend the coordinate on their end.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The apcoa api coordinates are in the middle of the road - which is not correct. The patch file takes the coordinates from apcoas map: https://www.apcoa.de/einmal-parken/standorte/muenchen/allianz-arena - which is a suitable location.

"restrictions": [
{
"type": "WOMEN",
"capacity": 53
Copy link
Collaborator

Choose a reason for hiding this comment

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

The capacity for women should be 35 according to the website - https://www.apcoa.de/einmal-parken/standorte/muenchen/p-r-froettmaning

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Right, this seems to be a number twister

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.

2 participants