Skip to content

Conversation

@fdcavalcanti
Copy link
Contributor

Description

This PR adds an article titled NuttX for Motor Control and Sensing: IMU Integration and Wireless Data Streaming.

This is the second and final part of a small series.

Note: I've moved this article to month 7 already, even though it is June 30th. Hope it is ok.

Related

Relates to #480

Testing

Build OK locally.


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@pedrominatel pedrominatel added the needs review Needs someone to be assigned to review label Jun 30, 2025
@pedrominatel pedrominatel requested review from FBEZ and f-hollow June 30, 2025 16:15
Copy link
Collaborator

@f-hollow f-hollow left a comment

Choose a reason for hiding this comment

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

@fdcavalcanti Thank you for contributing this excellent article!

I left a couple of suggestions and nitpicks. Otherwise, LGTM!

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 2205004 to 52d86c7 Compare July 1, 2025 23:54
@fdcavalcanti
Copy link
Contributor Author

@pedrominatel @f-hollow all good?
I'm thinking of opening a separate PR after this to update the first article with a link to this one. Sounds good?

@f-hollow
Copy link
Collaborator

f-hollow commented Jul 2, 2025

@pedrominatel @f-hollow all good? I'm thinking of opening a separate PR after this to update the first article with a link to this one. Sounds good?

@fdcavalcanti

Why not update the first article in this PR? It will be faster. You can create a separate commit, and we can merge it without squashing.

But if you think it should be done in a separate PR, please go ahead.

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 52d86c7 to 9b9ce26 Compare July 2, 2025 12:44
@fdcavalcanti
Copy link
Contributor Author

fdcavalcanti commented Jul 2, 2025

@f-hollow Done. Updated the first part with a link to this one.

@f-hollow
Copy link
Collaborator

f-hollow commented Jul 2, 2025

@fdcavalcanti Thank you very much!

@pedrominatel I have no more questions. PTAL and feel free to merge the article.

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 9b9ce26 to 5e2c6be Compare July 2, 2025 13:43
@pedrominatel pedrominatel added GitLab-Sync-Merge Contribution reviewed and synced from the internal GitLab mirror and removed needs review Needs someone to be assigned to review labels Jul 2, 2025
@pedrominatel pedrominatel merged commit 7f31540 into espressif:main Jul 2, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GitLab-Sync-Merge Contribution reviewed and synced from the internal GitLab mirror

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants