Skip to content

Linked place names to Default language #3721

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

Closed
wants to merge 3 commits into from

Conversation

tarhsandrasepiah
Copy link

Fixes #2714

Hello @lonvia , I have nominatim installed

image

please I made a few changes to nominatim , but I don't know how to view the effect of the change. May be I am not compiling nominatim correctly. please what are the steps to take after making changes.

@mtmail
Copy link
Collaborator

mtmail commented Apr 17, 2025

Add BDD tests first https://github.com/osm-search/Nominatim/blob/master/docs/develop/Testing.md

The github issue asked "Note: if you want to pick this up as a 'good first issue', please start with creating a BDD test that reproduces the example from the issue and create a PR with the test (which should still be failing at this point). Start with the following test as a blueprint: [...]"

Later you won't need to install Nominatim. It comes with a nominatim-cli.py script which run the code in the /src directory directly. https://github.com/osm-search/Nominatim/blob/master/docs/develop/Development-Environment.md That'll allow you to create a new database, import data and run searches against the imported data.

@lonvia
Copy link
Member

lonvia commented Apr 17, 2025

#3687 has added a test for this. The test has moved to

@skip
Scenario: Linked places expand default language names

Remove the @skip and run the test to see if it is successful. You can run just the one test with pytest test/bdd/test_db.py -k linked_places_expand_default_language_names.

@tarhsandrasepiah
Copy link
Author

please i get psycopg.errors.UndefinedFunction: function placex_insert() does not exist after running the test.

image

please how may I go about it.

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.

Linked place names need to include the default language
3 participants