Data warnings/errors about itsb HOT 5 CLOSED

sweeeet. go team

from itsb.

This is pretty much what we were doing in the old app. Just skipping these. There should no missing place names in theory. But that part has to happen at the data curation stage. We have tests over there.

from itsb.

@elotroalex Looks like 81 in the original Google Sheets does indeed have an error (Ouagadougou, maybe incorrect spelling?), but line 11 (no place name) does not.

Feel free to update the CSV in this repo with any changes, or let me know how we should edit them!

from itsb.

You can replace it with dummy data, or google maps. Ougadougou is the right spelling for the capital of Burkina Faso. It might just be missing from the georeferenced master data set. Back in the day, since this process was run on the terminal, i would get a list of place names that needed fixing, and I would go ahead and fix them. I'm just thinking ahead here rather than worrying about these examples. If we go the GitHub actions route, we need an error that spits these out on the browser CLI for GitHub, the one where you can see the tasks/process run. That would help address these in the future as they come

from itsb.

You can replace it with dummy data, or google maps

Fixed!

If we go the GitHub actions route, we need an error that spits these out on the browser CLI for GitHub, the one where you can see the tasks/process run. That would help address these in the future as they come

Indeed, I've kept the same CLI logging for this, and added some more for the country codes warnings. You'll see that if you run convertJSON locally, and it's also being logged in our Github Action.

Feel free to close this issue; I opened it mostly as reference for additional data cleanup to make it easier to reference than just the Action/CLI logs, but totally fine to close if that's not necessary.

from itsb.