Skip to content

Conversation

mcmchris
Copy link
Collaborator

@mcmchris mcmchris commented Apr 6, 2023

What This PR Changes

  • I developed and wrote an application note that showcases a project done with the Nicla Voice Arduino board.
  • I created a content.md and assets folder in the Docs-content/content/hardware/06.nicla/nicla-voice/tutorials/ei-intruder-detector.

Contribution Guidelines

mcmchris and others added 12 commits April 11, 2023 10:22
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
…uder-detector/content.md Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
@mcmchris mcmchris requested a review from jcarolinares April 11, 2023 14:48
Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

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

This content is ready for publiccation

@mcmchris `please do not merge

Once we have your GO @martab1994 we go live

Thank you all, amazing job!

Copy link
Contributor

@martab1994 martab1994 left a comment

Choose a reason for hiding this comment

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

Thanks so much for you great work! Just please have a look at my comments to highlight we are using Machine Learning Tools powered by Edge Impulse and avoid mentioning Edge Impulse since we have this white-label version. Thanks!

- [Arduino IDE 1.8.10+](https://www.arduino.cc/en/software), [Arduino IDE 2.0+](https://www.arduino.cc/en/software), or [Arduino Web Editor](https://create.arduino.cc/editor)
- If you are going to use an offline Arduino IDE, you must install the following libraries: `ArduinoBLE`, `ArduinoIoTCloud`, and `Arduino_ConnectionsHandler`. You can install them using the Arduino IDE Library Manager.
- The [intruder detector example code](assets/Nicla_Portenta_Codes.zip)
- [Edge Impulse® CLI](https://docs.edgeimpulse.com/docs/edge-impulse-cli/cli-overview), which you can install following these [instructions](https://docs.edgeimpulse.com/docs/edge-impulse-cli/cli-installation)
Copy link
Contributor

Choose a reason for hiding this comment

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

Here again, the right path should be from Arduino IoT Cloud, use the Machine Learning Tools integration, basically a white label version of the full Edge Impulse platform. See here an example on how we use this machine learning tools for Nicla Voice: https://docs.arduino.cc/tutorials/nicla-voice/getting-started-ml

Copy link
Contributor

@jcarolinares jcarolinares Apr 12, 2023

Choose a reason for hiding this comment

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

I'm afraid we do not have a white label or more elegant version of the edge impulse cli tool and therefore we have to mention it because it is a tool necessary to perform the project (I will check if we can do it differently in the future if possible)

However, we can improve the message following the same strategy done at the Nicla Voice Getting Started:

image

Copy link
Contributor

@jcarolinares jcarolinares Apr 12, 2023

Choose a reason for hiding this comment

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

Changed to

  • Edge Impulse® CLI ,to easily flash any Machine Learning model created with the Arduino Machine Learning Tools, we can install the Edge Impulse Cli tool following these instructions

I don't find a best solution with the actual status of the tools, should be enough imho cc @martab1994

@jcarolinares
Copy link
Contributor

jcarolinares commented Apr 12, 2023

@mcmchris I will do these changes thank you!

@jcarolinares
Copy link
Contributor

@martab1994 changes done.

Once you think is ready please change your review to approved it and tell me a publication date

Thank you for the priceless feedback!

Copy link
Contributor

@martab1994 martab1994 left a comment

Choose a reason for hiding this comment

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

LGTM thanks =) I think we can merge! Thanks so much for all your effort =)

Copy link
Contributor

@martab1994 martab1994 left a comment

Choose a reason for hiding this comment

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

Amazing!

@jcarolinares jcarolinares merged commit 052b74d into main Apr 13, 2023
@jcarolinares jcarolinares deleted the mcmchris/nicla-voice/app-note-intruder-detector branch April 13, 2023 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants