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

Max height quest doesn't allow inputting two-digit height values #5232

Closed
waldyrious opened this issue Sep 3, 2023 · 6 comments
Closed

Max height quest doesn't allow inputting two-digit height values #5232

waldyrious opened this issue Sep 3, 2023 · 6 comments
Labels

Comments

@waldyrious
Copy link
Contributor

waldyrious commented Sep 3, 2023

I'm not sure if this is a regression of #3000 or a new issue altogether, but I am again unable to enter large height values (10m or more). This time it's not due to a limitation in the total number of characters (as was the case in #3000), but rather due to only being able to input a single digit before the decimal point.

Here's a screen recording of me hitting this limitation:

output8.mp4
@waldyrious waldyrious added the bug label Sep 3, 2023
@waldyrious
Copy link
Contributor Author

For additional context: Here in Portugal, it is quite common, at least in the North, to have extremely high "clearances" under highway bridges simply due to the geography of the terrain. Here are two notes I left due to being unable to enter the correct values in the app: 3865957, 3865958. For illustration here's a Mapillary shot corresponding to the first note.

@matkoniecz
Copy link
Member

matkoniecz commented Sep 3, 2023

For illustration here's a Mapillary shot corresponding to the first note.

Is there a height limit sign there?

@waldyrious
Copy link
Contributor Author

Is there a height limit sign there?

Yes, of course — I couldn't obtain the height otherwise. It's here:

image

Thanks for the quick fix, @westnordost!

@matkoniecz
Copy link
Member

Yes, of course — I couldn't obtain the height otherwise. It's here:

Oh, I completely missed it (even when pointed out on photo I cannot really read it). Thanks, that makes sense to me now!

@waldyrious
Copy link
Contributor Author

even when pointed out on photo I cannot really read it

Yeah, I can't read it from the photo either :) but that was just an old image I found to illustrate the situation. I surveyed the info directly as a passenger in a recent car trip that passed under that bridge.

@waldyrious
Copy link
Contributor Author

For the record, there were a couple ways I added an incorrect maxheight to before I noticed this bug, in changeset https://www.openstreetmap.org/changeset/140555809. Fortunately someone alerted me to the implausible maxheights, and I removed them in https://www.openstreetmap.org/changeset/142592971.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants