Skip to content

Conversation

@fdcavalcanti
Copy link
Contributor

Description

This PR adds an article titled NuttX for Motor Control and Sensing.
This is the first part of it, with one more to follow in the next weeks.

Related

None.

Testing

Built file on hugo server 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.

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 1e8d7de to aa7b577 Compare April 30, 2025 18:44
@pedrominatel pedrominatel requested a review from f-hollow May 2, 2025 13:01
@pedrominatel pedrominatel added the needs review Needs someone to be assigned to review label May 2, 2025
@pedrominatel
Copy link
Member

Thanks @fdcavalcanti for your contribution!


## Introduction

This series of articles demonstrates how to use multiple ESP32-C6 peripherals to build a complete motor control and sensing system. We'll explore:
Copy link
Contributor

@sylvioalves sylvioalves May 5, 2025

Choose a reason for hiding this comment

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

I would suggest to avoid the use of "We/they/our" in overall documentation. @pedrominatel any recommendation about it or it is good as is?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would say this applies to academic articles. Many articles on the portal are "friendly".

Copy link
Member

Choose a reason for hiding this comment

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

I don't think using "we/they/our" is inappropriate for this content.

#define BASE_PPR 11 /* Base encoder PPR */
#define GEAR_RATIO 34 /* Gear reduction ratio */
#define PULSES_PER_REV (BASE_PPR * GEAR_RATIO)
```
Copy link
Contributor

@sylvioalves sylvioalves May 5, 2025

Choose a reason for hiding this comment

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

By looking at the review page here, sounds the formatting isn't ok here.. PTAL. Text color below is not the same color as in other parts of the document.

Copy link
Contributor Author

@fdcavalcanti fdcavalcanti May 5, 2025

Choose a reason for hiding this comment

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

Seems Markdown does not color the defines.

I added some functions and it highlighted fine on the same code block.

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from aa7b577 to 4eb5729 Compare May 5, 2025 13:07
@fdcavalcanti
Copy link
Contributor Author

Applied most of the suggestions and also added a schematic.

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 4eb5729 to 408cc5d Compare May 5, 2025 13:09
@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch 2 times, most recently from 56d991c to 0c7e707 Compare May 6, 2025 12:00
@fdcavalcanti
Copy link
Contributor Author

Seems we have an error regarding the directory of the article. Should I change it?

@pedrominatel
Copy link
Member

Seems we have an error regarding the directory of the article. Should I change it?

Yes, you need to move your article folder to the content/blog/2025/05.

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 0c7e707 to cd28c61 Compare May 7, 2025 17:20
@fdcavalcanti
Copy link
Contributor Author

@pedrominatel done

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from cd28c61 to 72c3279 Compare May 14, 2025 14:18
@fdcavalcanti
Copy link
Contributor Author

Thank you @f-hollow

@fdcavalcanti fdcavalcanti force-pushed the feature/motor-control-sensing branch from 72c3279 to 284fed1 Compare May 16, 2025 10:26
@pedrominatel pedrominatel merged commit b632410 into espressif:main May 16, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review Needs someone to be assigned to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants