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

NRW vDOP is blank while listed as best imagery for at least some areas #2273

Open
matkoniecz opened this issue Apr 3, 2024 · 11 comments
Open

Comments

@matkoniecz
Copy link
Contributor

https://www.openstreetmap.org/edit?editor=id#map=19/51.18137/7.17874

screen02

@tordans
Copy link
Contributor

tordans commented Apr 13, 2024

See also #2262

@crackwitz
Copy link
Contributor

crackwitz commented Apr 26, 2024

It's been completely blank for weeks because all the imagery is now in the proper "NRW Orthophoto". They removed "done" areas from vDOP as they worked on them.

Today it's even the default choice. "NRW Liegenschaftsregister" used to be the default but it loads really slowly, so that shouldn't be the first choice either.

"NRW Orthophoto" should be top choice. It loads quickly and it's up to date.

I wish iD would remember my choices and other settings across sessions...

Unless someone else is working on it, I'll swap the "best" properties of NRW vDOP and NRW Orthophoto, then open a PR.

crackwitz added a commit to crackwitz/editor-layer-index that referenced this issue Apr 26, 2024
@tordans
Copy link
Contributor

tordans commented Apr 26, 2024

Offtopic:

I wish iD would remember my choices and other settings across sessions...

FYI: If you use the stand alone version from the readme in GitHub you get proper url parses that store the selected imagery. You can then bookmark that.

@crackwitz
Copy link
Contributor

Thank you for that hint. I gather that involves running it locally. I might try that. I'm already redirecting tile requests through a local nginx because the NRW WMS tells clients to not cache at all, which I find silly... i.e. I'm already somewhat into running things locally.

I just noticed that this issue here might have been created with the intention of perhaps removing the vDOP entry. I have no strong feelings on that either way.

@tordans
Copy link
Contributor

tordans commented Apr 26, 2024

Thank you for that hint. I gather that involves running it locally. I might try that.

No, you can use https://ideditor-release.netlify.app/#background=nrw_ortho_wms&disable_features=boundaries&map=18.14/50.77696/6.08326 (from https://github.com/openstreetmap/id?tab=readme-ov-file#participate)

@tordans
Copy link
Contributor

tordans commented Apr 26, 2024

Back to the issue at hand: I think @pathmapper it would be best if you took over here since you introduced the data in #2022.

We have a few things fix here, IMO

I did no research in this area other than scanning the comments in #2022 (comment) and similar issues.

But I think we have to improve the situation soon by…

  • if possible, change the area-definition to the area where images are actually present; your comment above could mean that this might be possible. But should the area change constantly, that would not work.
  • we need to pick a "best" layer that shows data for all the area that is defined in the area-defintion. Everything else is just bad UX for users. It is great if there is possibly newer data, but that has to be a secondary step, selected by mappers that know about those things.

Given those constrains, what would you say is the best way forward, @pathmapper

@pathmapper
Copy link
Contributor

pathmapper commented Apr 27, 2024

I don't have much to add to #2022 (comment):

It all depends on how best is definied, and IMHO there is currently no such clear definition. So this should be resolved.

I always understood best as most recent available data, because currentness of data is crucial if one wants to trace things for OSM.

if possible, change the area-definition to the area where images are actually present; your comment above could mean that this might be possible. But should the area change constantly, that would not work.

The area is changing constantly, e.g. for
https://ideditor-release.netlify.app/#background=nrw_idop_wms&disable_features=boundaries&map=18.56/51.34159/6.33003

nrw_idop_wms is currently more recent than nrw_otho_wms (link)

image

we need to pick a "best" layer that shows data for all the area that is defined in the area-defintion.

Do we really need to pick a "best" layer? For NRW we are having three different aerial sources, and it's constantly changing which source of them is the most recent for a given area.

Given the lack of a clear definition for what is "best", I currently won't assign "best" to any of the three sources.

@crackwitz
Copy link
Contributor

crackwitz commented Apr 27, 2024

I think that casual users, entering the editor, should never be faced with a blank background layer. That is just a bad user experience. And it's annoying to everyone using the editor.

Those vDOP and iDOP layers are "incomplete" and constantly changing. Even worse, the vDOP layer "lost" content over time (as it was done processing and incorporated into "NRW Orthophoto", the normal DOP), so if one saw imagery with the default behavior yesterday, and then opened the editor again today, and it was blank where it showed something yesterday, that is utterly bewildering.

Those should definitely not be the default picks.

They should remain though, with a description of what to expect of those layers, and what not to expect.

One could also "advertise" those layers in the description of the regular "NRW Orthophoto".

I did not expect my PR to be considered controversial.

@crackwitz
Copy link
Contributor

crackwitz commented Apr 27, 2024

Given the lack of a clear definition for what is "best", I currently won't assign "best" to any of the three sources.

What happens if none of the NRW sources are tagged as "best"?

If there are no other "best" sources, we'd get Bing Maps because it's alphabetically the first.

If there are other "best" sources, as happens near the borders and sometimes a sizable distance "inland", then those take precedence, but they're blank in NRW.

A simple and generally beneficial change is to "tolerate" dop (NRW Orthophoto) as "best". It is the best choice in general, considering the downsides of vdop (completely blank!) and idop (blurry, 50% coverage).

@pathmapper
Copy link
Contributor

If there are no other "best" sources, we'd get Bing Maps because it's alphabetically the first.

True, that's also not nice. But at least there's no layer advertised as best which is outdated for large areas.

I don't have a strong opinion here, go with dop as best if you want.

Just keep in mind that it's expecteded that idopwill get 100% coverage this year so you will have two sources covering 100% of NRW but which are each only the most recent for ~50% compared to the other one.
(btw the "blurryness" is due to the fact that idop has 20 cm ground resolution compared to dop which has 10 cm)

Also new vdop will be available soon which then will be ~2 years more recent than dop.

I really do like the comment from #130 (comment):

best was a good enough hack when we really only had Bing, Mapbox, and whatever local imagery to choose from. We have so many more imagery to trace from now that we need something a bit more nuanced (this is a great problem to have!)

So it's time for something new, looks like #130 is a good place for anyone who has some suggestions to join the discussion.

@matkoniecz
Copy link
Contributor Author

I always understood best as most recent available data, because currentness of data is crucial if one wants to trace things for OSM.

Resolution also matters (extreme example: daily updated satellite imagery with resolution of 100m is worse than aerial with resolution of 1cm updated 6 months ago).

Quality of georeferencing, colours, when it was taken (leafless imagery) may also matter.

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

No branches or pull requests

4 participants