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

Add aerial imagery, fixes #56 #2755

Merged
merged 16 commits into from Apr 19, 2021

Conversation

matkoniecz
Copy link
Member

Notes:

  • it uses Esri aerial as it has a global coverage and - unlike Bing - is available without byzantine setup.
  • it works, but I suspect that it is architecturally sad way of getting preferences.
  • I worry a bit about this setting encouraging people to start and answering remotely and guessing "gray on aerial"? definitely asphalt! But it would be useful in some cases.
  • fixes second-oldest open issue ( https://github.com/streetcomplete/StreetComplete/issues?q=is%3Aissue+is%3Aopen+sort%3Acreated-asc ) and finishes branch that I started a year ago :)

@andrewharvey
Copy link
Contributor

Awesome. In the future it would be nice to use the "best" layer from Editor Layer Index, but that would be a fair bit more effort, so glad to see this launch first.

@HolgerJeromin
Copy link
Contributor

Please add
fixes #56
To the first comment to be recognised by the PR gui on the right/bottom.

@matkoniecz
Copy link
Member Author

To the first comment to be recognised by the PR gui on the right/bottom.

It is in the title what should be sufficient.

And likely is in one of commits already anyway.

@matkoniecz
Copy link
Member Author

Awesome. In the future it would be nice to use the "best" layer from Editor Layer Index, but that would be a fair bit more effort, so glad to see this launch first.

I worry a bit about potential misuse. So sinking time into implementing all that seems to not be a good idea if it could be potentially removed anyway.

And many imagery sources require various special support, API keys, workarounds etc so it would be quite time consuming.

Copy link
Member

@FloEdelmann FloEdelmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's surprisingly little code for such a big feature!

app/src/main/res/values/arrays.xml Outdated Show resolved Hide resolved
@matkoniecz
Copy link
Member Author

That's surprisingly little code for such a big feature!

I never though that I will praise Esri for making the simplest product that is available for free. But I must say that from worldwide aerial views without extra complexity - their is the best among ones available for OSM mapping.

Also, nice case of how measuring programming work by lines of code is a bad idea: that was not simple to figure out how to run it (commit history reveals it only partially).

matkoniecz and others added 2 commits April 17, 2021 12:44
Co-authored-by: Flo Edelmann <florian-edelmann@online.de>
@westnordost
Copy link
Member

Doesn't ESRI require attribution in the corner of the map or something?

@matkoniecz
Copy link
Member Author

No, see #56 (comment)

@westnordost westnordost merged commit 3da55f0 into streetcomplete:master Apr 19, 2021
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.

None yet

5 participants