Skip to content

Adding Edge Impulse + micro-ROS demo#389

Merged
ralph-lange merged 7 commits into
micro-ROS:masterfrom
avibrown:master
Sep 30, 2022
Merged

Adding Edge Impulse + micro-ROS demo#389
ralph-lange merged 7 commits into
micro-ROS:masterfrom
avibrown:master

Conversation

@avibrown
Copy link
Copy Markdown
Contributor

Hi @ralph-lange ,

Thanks for the invitation to add this demo page. I hope I formatted everything correctly. I will be in touch when I add more examples / additional board support to coordinate with you whether to update this demo page.

Copy link
Copy Markdown
Contributor

@ralph-lange ralph-lange left a comment

Choose a reason for hiding this comment

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

Thank you very much for contributing this demo/tutorial. The description is very well done and concise. With two somewhat longer sentences, readability can be increased by a comma, I think. But I am not a native speaker.
Also, you may omit the _demo' in the folder name since the parent folder is already named demos. (I know, all other folders have this unnecessary _demo`.)

Comment thread _docs/tutorials/demos/edgeimpulse_demo/index.md Outdated
Comment thread _docs/tutorials/demos/edgeimpulse_demo/index.md Outdated
avibrown and others added 3 commits September 30, 2022 09:38
Co-authored-by: Ralph Lange <ralph-lange@users.noreply.github.com>
Co-authored-by: Ralph Lange <ralph-lange@users.noreply.github.com>
@avibrown
Copy link
Copy Markdown
Contributor Author

I changed the folder name as requested :) Let me know if there is anything else. Thanks.

Copy link
Copy Markdown
Contributor

@ralph-lange ralph-lange left a comment

Choose a reason for hiding this comment

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

Perfect. Thanks again very much!

@ralph-lange ralph-lange merged commit c996a0c into micro-ROS:master Sep 30, 2022
@avibrown
Copy link
Copy Markdown
Contributor Author

avibrown commented Sep 30, 2022

I realized I hadn't changed the name of the folder within the header as well, so I did that now. I don't know if that is the reason why the build failed..?

Edit: Sorry, now I see that you merged the most recent commit. My mistake.

@ralph-lange
Copy link
Copy Markdown
Contributor

The build failed because of few broken links in other pages of the website - not in your page.

@avibrown
Copy link
Copy Markdown
Contributor Author

@ralph-lange sorry to be a nuisance - but shall I also update overview/index.md so that the table includes my addition? Also, is there anything I need to do so that the demo appears in the "Demos" dropdown menu on the site?

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