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

Update ifopt script #72

Closed
wants to merge 3 commits into from
Closed

Conversation

lonvia
Copy link
Contributor

@lonvia lonvia commented Nov 21, 2024

Updates the IFOPT update script for recent versions of Photon. This includes some fixes and improvements:

  • update to recent psycopg3
  • make imported address columns optional and configurable
  • copy names from IFOPT data into Nominatim database when Nominatim does not have a name
  • fix issue where IFOPT was not set when Nominatim does not have any extratags yet
  • be more clever about marking data as changed (no need for the '-i' flag anymore)
  • increase the minimum ID for external data (hopefully now works for more than a year)
  • allow to skip CSV lines according to match_state
  • do not set name and alt_name from IFOPT data when they are the same

Running against the current DELFI CSV, I get:

Matched: 390737, updated: 0, added: 77022

Please note that the update method of Photon has changed. You must set up Photon for updates with -nominatim-update-init-for before running the script. I've updated the update-nominatim script to follow the new steps for running updates.

Further note that I have updated the name of the 'GlobaleID' column in the CSV which likely means the script cannot be run against the NVBW data anymore.

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