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

[PXCT-254] Motor Anomaly Detection Opta + Nicla Sense ME (App Note) #2346

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

DavidsonMSD
Copy link
Collaborator

Complete application note

What This PR Changes

  • All the written text and images

Contribution Guidelines

Update of images and completion of the written section
@mcmchris mcmchris changed the title Complete application note [PXCT-254] Motor Anomaly Detection Opta + Nicla Sense ME (App Note) Jan 7, 2025
Copy link
Collaborator

@mcmchris mcmchris left a comment

Choose a reason for hiding this comment

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

Reviewed and updated. LGTM. Thanks @DavidsonMSD

Copy link

github-actions bot commented Jan 7, 2025

Preview Deployment

Waiting for deployment to complete...

@@ -0,0 +1,713 @@
---
title: 'ML for Motor Anomaly Detection with Nicla Sense ME & Opta™'
Copy link
Contributor

Choose a reason for hiding this comment

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

Hello, I have a small proposal @DavidsonMSD and @mcmchris for different reasons

Suggested change
title: 'ML for Motor Anomaly Detection with Nicla Sense ME & Opta™'
title: 'Edge AI for Motor Anomaly Detection with Opta™ & Nicla Sense ME'

I know length it is important but ML is not a good idea for SEO and clarity reasons. I also put Opta first being more important for our users for this example.

Thank you for the hard work it is really great!

…aly-detection-app-note/content.md

Co-authored-by: Julián Caro Linares <jcarolinares@gmail.com>
@mcmchris
Copy link
Collaborator

mcmchris commented Jan 7, 2025

@jcarolinares thanks, let me check how I can meet the 60 char limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants