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

Poor focus in snapshots from athena #1669

Open
csouers opened this issue Jun 9, 2020 · 1 comment
Open

Poor focus in snapshots from athena #1669

csouers opened this issue Jun 9, 2020 · 1 comment

Comments

@csouers
Copy link
Contributor

@csouers csouers commented Jun 9, 2020

Example snapshots

Still from driving video

Describe the bug
Focus appears to be off significantly in rear camera snapshots

How to reproduce or log data
Capture snapshot via athena.

Expected behavior
Snapshot with good focus.

Device/Version information (please complete the following information):

  • Device: comma two
  • Dongle ID: 95e637a323d6e9fc
  • Version: master-ci branch based off of 9d8ff2887c280388eb0abc589d5f562a6dfac0ad
  • Car make/model: Civic Hatchback 2017

Additional context
Focus during normal driving footage is fine and looks great. Only the snapshots appear to be affected. I query snapshots constantly throughout the day and it always exhibits the same behavior. I thought maybe it was compression at play, but I'm starting to thing that it's not the issue.

@csouers csouers changed the title Poor focus in snapshots Poor focus in snapshots from athena Jun 9, 2020
@pd0wm
Copy link
Contributor

@pd0wm pd0wm commented Jun 9, 2020

It probably doesn't wait long enough for camerad to be completely focussed. Does it help if you increase the 3 second timeout to something large like 10 seconds?https://github.com/commaai/openpilot/blob/master/selfdrive/camerad/snapshot/snapshot.py#L41

Camerad now outputs a sharpness score. We should probably change the snapshot code to subscribe to that and detect when the image is properly focused before taking the snapshot (obviously with some max time).

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.

Follow Lee on X/Twitter - Father, Husband, Serial builder creating AI, crypto, games & web tools. We are friends :) AI Will Come To Life!

Check out: eBank.nz (Art Generator) | Netwrck.com (AI Tools) | Text-Generator.io (AI API) | BitBank.nz (Crypto AI) | ReadingTime (Kids Reading) | RewordGame | BigMultiplayerChess | WebFiddle | How.nz | Helix AI Assistant