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

Frauentag in Berlin doesn't evaluate as PH? #458

Open
tohaklim opened this issue Mar 8, 2024 · 2 comments
Open

Frauentag in Berlin doesn't evaluate as PH? #458

tohaklim opened this issue Mar 8, 2024 · 2 comments
Labels
status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation. type: bug A confirmed report of unexpected behavior.

Comments

@tohaklim
Copy link

tohaklim commented Mar 8, 2024

From what I could see in the code, location to be applied to PH/SH evaluation is based on reverse geocoding of coordinates which are fed to Nominatim, not on the hardcoded tag values of the POI itself.
Yet when I wanted to check this REWE on 8th March, expecting it to be closed due to PH off, it was shown as open.
I played around with the validation, and got eg the following error when inserting SH
There are no holidays (SH) defined for country de and state {{state}}.

Does that mean the PH evaluation fails because reverse geocoding can't determine the state, despite having the country?

@jaller94
Copy link

To reproduce:

https://openingh.ypid.de/evaluation_tool/?EXP=Mo-Sa%2007%3A00-23%3A30%3B%20PH%20off&lat=52.55048&lon=13.409781&mode=0&DATE=1709887260000

Yes, Nomatin does not return address.state which is used by the website.

{
  // other attributes
  "address":{
    "city":"Berlin",
    "ISO3166-2-lvl4":"DE-BE",
    "country":"Deutschland",
    "country_code":"de"
  }
}

@ypid
Copy link
Member

ypid commented Mar 17, 2024

Frauentag is defined as PH for Berlin.

It is as @jaller94 wrote.

Someone please clarify if that is expected behaviour of Nominatim. If not, this library needs to be updated.

@ypid ypid added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation. type: bug A confirmed report of unexpected behavior. labels Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation. type: bug A confirmed report of unexpected behavior.
Projects
None yet
Development

No branches or pull requests

3 participants