-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
UI - Show max speed sign next to speed camera icon #3721
Comments
Great Idea ! |
👍 |
I see several problems here:
|
Perhaps the max allowed speed widget could increase its size during the camera alert. The idea is to quickly identify the speed limit of the camera. I didn't understand the problem with variable speed limit areas. Shouldn't the speed_camera have the lowest maxspeed limit? About the traffic light camera alert, of course that nobody wants to encourage illegal driving. Sometimes the camera is a just a trap to collect fines, for example when the yellow light duration time is too short or the light changes and the car stops a few centimeters over the pedestrian crossing. Think of this alert as if it were a sign in the street announcing the camera: |
We have a motorway tunnel which normally is 80 km/h and has cameras. The limit can be reduced to 60 or 40 depending on traffic condition or hazards in the tunnel. When the signs are changed, the camera will flash also when you exceed the temporary lower limit. There is some grace time so only vehicles who have seen the lower limit when entering the tunnel are flashed. As for trapping with yellow being to short, that should be illegal and you should complain to your court if that happens. Here, there are prescribed yellow phases depending on the allowed speed (i.e. a 70 road has longer yellow than 50) that allows to react humanly. |
+1 for the proposal to show a max speed sign and a traffic light sign on the screen. |
Speedcam limit is also should be announced. |
- RouteCalculationResult attachAlarmInfo method extended to add max speed to speed camera AlarmInfo taken either from speed camera point (preferred) or from RouteSegmentResult. - WaypointHelper getMostImportantAlarm and announceVisibleLocations updated to attach actual distance to speed camera AlarmInfo. - AlarmWidget extended with ability to show max speed for speed camera when available. - VoiceRouter.announceAlarm extended with special handling for speed camera announcement to announce distance and max speed when available. - CommandBuilder / JsCommandBuilder extended with new speedCameraAlarm method, which allows announce speed camera with distance and max speed. - TestVoiceActivity extended to test speed camera with distance and max speed announcement.
- Rework icons to be more consistent with american signs (separated icons added for american region) and less confusing with regular speed limit. - Distance to camera visualization added. - Additionally fix alarm icon re-sizing on large screens (related not only to speed cameras, icon re-sizing was missed, as result bottom text was out of icon area)
- Add fallback for C_SPEED_CAMERA_ALARM in JsCommandBuilder - Combine checks for text change / bottom text change in the AlarmWidget.updateInfo into one conditional statement and extract updateTextWidget / updateBottomTextWidget methods.
- Refactor AlarmInfo to always accept maxSpeed in meters per second, and provide it either in meters per second or in accordance with requested speed constants settings.
Hello,
It would be great if OsmAnd could show the max speed sign next to the speed camera icon, reading the speed limit value from the maxspeed tag in the enforcement=maxspeed relation or from the highway=speed_camera node (device).
It could also show a traffic light sign for the enforcement=traffic_signals relation.
I made two mockups:
Thanks!
Ignacio
The text was updated successfully, but these errors were encountered: