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

Updates face detection to sdk 4.8.0.2 #113

Merged
merged 2 commits into from
Mar 3, 2023

Conversation

hmllr
Copy link
Contributor

@hmllr hmllr commented Feb 17, 2023

  • changes to CPX messages for streaming
  • adds configurable streaming resolution
  • adds missing viewer script
  • removes read input from file, this was broken (can be resurrected by fetching the depreciated frame streamer code from the sdk, but is not worth it)
  • there are remaining issues: camera acquisition fails more often when a) wifi is enabled and not yet conncted b) the cluster frequency is high (75MHz seems pretty stable, 100MHz is still ok) c) the image is very dark (no clue why, but can almost be used for obstacle detection...)
  • camera test is also working with 4.8.0.2 (only ASYNC capture though)

- changes to CPX messages for streaming
- adds configurable streaming resolution
- adds missing viewer script
- removes read input from file, this was broken
(can be resurrected by fetching the depreciated frame streamer code from the sdk, but is not worth it)
- there are remaining issues: camera acquisition fails more often when
	a) wifi is enabled and not yet conncted
	b) the cluster frequency is high (75MHz seems pretty stable, 100MHz is still ok)
	c) the image is very dark (no clue why, but can almost be used for obstacle detection...)
- camera test is also working with 4.8.0.2 (only ASYNC capture though)
@hmllr
Copy link
Contributor Author

hmllr commented Feb 17, 2023

addresses #41 and #92 , maybe also #106

Copy link
Member

@knmcguire knmcguire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! In the documentation I would prefer to use cfloader flashing instead of using the stlink... but that might be difficult with the camera test.
I'll make a seperate PR for that

@knmcguire knmcguire merged commit 915603a into bitcraze:master Mar 3, 2023
@knmcguire knmcguire added this to the next-release milestone Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants