Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updates I made to make it run + documentation/debugging #11

Merged
merged 7 commits into from
Mar 12, 2025

Conversation

wil-liammacleod
Copy link
Contributor

Added DEBUG_MODE, and conditional prints if the value is true
Added new pinout for ESP32 S3, Expainded documentation for Pico and ESP pinout
Tested on Micro python 1.24, with ESP32 S3 Wroom-1, with Arducam "MEGA-3MP 10th ANNIV. EDITION"
Updated readme

@wil-liammacleod
Copy link
Contributor Author

I also added chrisrothwell1's code from their zip file.
#9 (comment)

The burst save function worked for resolution = '640x480', and save much faster, 31.373s vs 0.8679999s
But note that for some reason if I try to save a resolution = '320X240' the output file is empty

@CoreProduction4
Copy link
Contributor

Hi Liam,

Thank you very much for contributing!

All of these updates are very much appreciated, we're strapped for time at the moment but hope to have this merged soon!

Liam

@CoreProduction4
Copy link
Contributor

Hi Liam,

I just had the chance to test this and its great! I'll look at getting this merged and the documentation updated to reflect the repos much more mature state (Thanks to you and Chris's work).

Thanks again for your work on this repo and stay tuned for a merge.

Liam

@CoreProduction4 CoreProduction4 merged commit 837b155 into CoreElectronics:main Mar 12, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants