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

Dynamic or Zoom based on speed #175

Open
Archdoog opened this issue Aug 4, 2024 · 4 comments
Open

Dynamic or Zoom based on speed #175

Archdoog opened this issue Aug 4, 2024 · 4 comments
Assignees
Labels

Comments

@Archdoog
Copy link
Collaborator

Archdoog commented Aug 4, 2024

At speeds above 25 mph/40 kph, our zoom level is too high to see upcoming maneuvers. I found that zooming out 2 increments on the button was the most ideal during several hours of driving on android. We should match iOS to this if needed.

Solution:

Subtract 2 from the default android zoom level and make sure iOS matches that new value.

@ianthetechie
Copy link
Contributor

This will be a lot nicer with #89.

@ianthetechie
Copy link
Contributor

@Archdoog this was expressed relative to some point in time whose values I no longer remember ;) Can you have a look again at where the defaults are relative to "optimal?"

@Archdoog
Copy link
Collaborator Author

Archdoog commented Oct 22, 2024

In some PR I adjusted android (and probably matched iOS, though not sure). It's quite nice currently for automotive at around 40-50 mph and passable for lower and higher speed. This current Android value should absolutely remain as is for the fixed default automotive camera.

I'd say for dynamic we could probably zoom 1 or 2 increment in below 35 mph and out above 65 mph. I would expect this to need revision, but testing those values will probably illustrate pretty well if the values are ideal after some use.

Once we define a decent range we can decide if it's good or needs to be fancier.

@ianthetechie
Copy link
Contributor

Sound good; thanks! I'll update the title on this ticket and start thinking about the interface for more dynamic customization.

@ianthetechie ianthetechie changed the title Dynamic or Zoom out automotive default. Dynamic or Zoom based on speed Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Up next
Development

No branches or pull requests

2 participants