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

comment out the initial country check #53

Closed
wants to merge 2 commits into from
Closed

Conversation

MurathanKurfali
Copy link
Collaborator

The pycountry-based initial country check is commented out.

@i-be-snek
Copy link
Collaborator

I thought about this for a bit... the issue was that we used to get a long list of locations where countries are mixed with area names.

@liniiiiii is this still the case?

If yes, we will then replace the method simple_country_check with either:

  • (A) a lookup method that uses UNSD and GADM instead -- one problem here is that GADM and UNSD only have the official names.
  • (B) a lookup method that shoots a request to OpenStreetMap to see if a Location exists as a "country" level administrative areas.

If no, then we simply drop these lines altogether and merge with main

@MurathanKurfali
Copy link
Collaborator Author

Whichever method we decide to continue with, I think it would be nice to run it on the gold data as well to make sure that everything we have there is classified as "country".

@liniiiiii
Copy link
Collaborator

Whichever method we decide to continue with, I think it would be nice to run it on the gold data as well to make sure that everything we have there is classified as "country".

Yeah, I plan to ask how to proceed this as well, let me run the dev set in the parse_events.py I revised for this function now, and to see if the code works!

@i-be-snek
Copy link
Collaborator

@liniiiiii so just to ask you again, we will have lists with mixed locations (countries and non-countries), is that correct?

@liniiiiii
Copy link
Collaborator

@liniiiiii so just to ask you again, we will have lists with mixed locations (countries and non-countries), is that correct?

@i-be-snek , yes, oh, no, really delayed response!

@i-be-snek
Copy link
Collaborator

Great, thanks @liniiiiii, good to know :)

@i-be-snek
Copy link
Collaborator

i-be-snek commented Sep 8, 2024

Since @liniiiiii is saying we will not have columns where country and location names are mixed, this function will be deprecated in #81

Unless I understood you incorrectly? Because your sentence had both a yes and a no?

@liniiiiii
Copy link
Collaborator

Since @liniiiiii is saying we will not have columns where country and location names are mixed, this function will be deprecated in #81

Unless I understood you incorrectly? Because your sentence had both a yes and a no?

My bad!!!! Yes is for the question, no is for my sorry of the late response :(

@i-be-snek i-be-snek closed this Sep 27, 2024
@i-be-snek i-be-snek deleted the country-fix branch January 17, 2025 13:36
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.

3 participants