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

classification example to freertos with cpx #89

Merged
merged 10 commits into from
May 9, 2022
Merged

Conversation

knmcguire
Copy link
Contributor

These are efforts to move the classification example to freertos and run CPX debugging instead of printf.

There is still an issue though, as it blocks after the first cpx debug to console. After GAP8:" *** Classification ***" I don't see "Starting to open camera". So it's blocking in between somehow?

@knmcguire
Copy link
Contributor Author

This line needs to be changed too which might interferes with the wifi example

This needs to be changed to

#define TXQ_SIZE (80)

There seems to be some cpx handling that needs to be changed first. See this issue: bitcraze/aideck-esp-firmware#9. Or we have to make a different initialization function for CPX where we can define the size.

@knmcguire
Copy link
Contributor Author

Or we should pick a smaller network than mobile netv2 just for the example, or try to see if we can reduce it further with quantizing the network even more? We need a bit more space for cpx to work :)

@knmcguire knmcguire marked this pull request as ready for review May 9, 2022 10:55
Copy link
Member

@evoggy evoggy 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, just some small stuff with formatting.

lib/cpx/src/com.c Outdated Show resolved Hide resolved
examples/ai/classification/classification.c Outdated Show resolved Hide resolved
examples/ai/classification/Makefile Outdated Show resolved Hide resolved
@knmcguire knmcguire merged commit d0b1c7b into master May 9, 2022
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