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

Quests counted in wrong country #1940

Closed
ahstro opened this issue Jul 6, 2020 · 5 comments
Closed

Quests counted in wrong country #1940

ahstro opened this issue Jul 6, 2020 · 5 comments
Labels
wontfix idea rejected because it is out of scope or because required work is not matching expected benefits

Comments

@ahstro
Copy link

ahstro commented Jul 6, 2020

I was browsing places from previous trips - Poland, USA, and England - and ended up mapping some places in England. For a while, they showed up as a British flag under My Profile > By Country, but now they're all logged as having happened in Poland. I don't remember if I actually mapped anything in Poland. Assuming this has to do with how StreetComplete groups edits? This is most likely the offending changeset.

How to Reproduce

  1. Probably map in one country
  2. Map in a different country soon enough for changes to be included in the same changeset

Versions affected
21.0

@ahstro ahstro added the bug label Jul 6, 2020
@westnordost
Copy link
Member

Hui, how did this changeset come together?

@westnordost westnordost added the feedback required more info is needed, issue will be likely closed if it is not provided label Jul 6, 2020
@ahstro
Copy link
Author

ahstro commented Jul 7, 2020

Like I said: just scrolling the map to places I have been recently, looking for easy things like road surfaces to add from memory. Bad practice, I know, I'll be better in the future 😇, but I'm assuming still a possible bug if someone is e.g. biking across country borders. Maybe uploads to OSM should be grouped according to country or something? (I'm just guessing about the internals here)

@westnordost
Copy link
Member

westnordost commented Jul 7, 2020

Mh, okay. Well, for the statistics, the app does not count each change but each changeset individually. It just takes the center point of the bounding box of a changeset to determine the country.

So for example if you solved 1000 address quests in Paris and 1 in Krakow within one changeset, all 1001 solved quests would count as having been solved in Germany.
But since such changesets are bad practice anyway (not only in context of StreetComplete) and since it only messes up the offending user's own statistics, I don't see the necessity to increase the complexity of the implementation by determining the country of each change individually. If this is very important to you, you can create a PR here that would implement such a feature https://github.com/westnordost/sc-statistics-service

@westnordost westnordost added wontfix idea rejected because it is out of scope or because required work is not matching expected benefits and removed bug feedback required more info is needed, issue will be likely closed if it is not provided labels Jul 7, 2020
@CalliOSM
Copy link

CalliOSM commented Jul 7, 2020

How do You handle changeset centroids that land outside of country borders? E.g. add the flag of the United Nations?
A larger part of my changesets for example could not be associated with a country.
https://nx4521.your-storageshare.de/s/gjFj5fAxA4pqx68

@westnordost
Copy link
Member

They are not handled at all. Actually, they should not appear in the by-country statistics at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix idea rejected because it is out of scope or because required work is not matching expected benefits
Projects
None yet
Development

No branches or pull requests

3 participants