-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
drag to adjust imagery offset #1340
Comments
+1 |
Entering values would also be fine. |
@kepta Do you think you could build something like this? Dragging anywhere in the grey |
Yes, this one is interesting.
|
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
Conversation from IRC regarding the issue.
|
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
A GIF might be more explaining. |
Hey that looks pretty good start!! Some ideas to try:
|
How do people usually talk about offset values? This is something I'm still a little confused about. Is this how people refer to offsets?
|
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
@kepta "Grey area" works for me but wouldn't some "tool" button be more intuitive? Or some checkbox? So that dragging the map wouldn't drag all objects, but only the image layer. It looks awesome in either case! @bhousel I think it doesn't matter as long as a particular editor uses consistent units. In addition, there's an offset database: http://wiki.openstreetmap.org/wiki/Imagery_Offset_Database which is #1124. So use whatever units you like, because we'll share them either through database (you'd need to recalculate to degrees) or just mention the editor name. If you want same units as in JOSM, you'd need to read its source code to figure out an algorithm. |
Yes, I think this is what @kepta built.. It looks like only the imagery layer is moving.
Ok, I was looking here: http://learnosm.org/en/josm/correcting-imagery-offset/ |
I think you should only care about what's stored in the database. Those units doesn't mean anything important. That could be pixel offset, recalculated each time you zoom. Or it can be meters. Whatever. And they can vary greatly, depending on the projection. iD only supports Web Mercator, so that's irrelevant. There are two kinds of objects in the offset database: imagery offsets and calibration objects. Imagery offsets are specific to imagery layer. So if you'd just store "city part", it'd mean anything. You must specify imagery layer too, like "New York Downtown, Bing". Additionally, there's a date, because offsets can change over time (new imagery available). Calibration objects are more interesting: it's basically known GPS coordinate of a specific feature. Like "GPS coordinates of the center of torch of the Statue of Liberty". Or "The center crossing of Wall Street". They don't require imagery specification, because those are objects easily identifiable visually. So it's the same as GPX traces: calibration objects are used to infer offsets for different imagery layers and store new imagery offset values. P.S. Statue of Liberty might not be a good example, because there's image perspective involved. |
Ok, if the units don't matter, we'll probably just go with something sane, like meters. |
I see you're doing the same ;) UTM coordinates are meters. |
😆 Last night I almost asked you, and instead decided to look it up on Wikipedia and got totally confused. I just searched again and found this guide to UTM coordinates that describes them much better! |
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
This patch allows to adjust imagery offset by 1. dragging 2. manually entering offset 3. fine tuning with buttons (closes openstreetmap#1340)
if the imagery is really off, clicking the arrows can be really tedious. i understand if this is intended though to prevent people getting totally wild with the offsets though, could go either way.
The text was updated successfully, but these errors were encountered: