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

Added a new Example : Servo Stop #45

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Chester-King
Copy link

Greetings,
I am Madhur Dixit, a third year student pursuing Bachelors degree in Computer Science and Engineering from Vellore Institute of Technology, Vellore, India.

I am looking forward to making valuable contributions to the Portenta repositories by writing MicroPython examples. Since Portenta board is not released yet I still have practiced with MicroPython on my NodeMCU. I am also planning on porting Arduino examples to Portenta and thus testing my MicroPython example codes on NodeMCU.
You can refer my work till now here:
https://github.com/Chester-King/NodeMCU-Micropython-Examples

Here is a simple PR which adds an example "servo-stop" which depicts the use of Servo with an Ultrasonic sensor. For convenience I have already Auto-Formatted the code.

Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

To make the example consistent with the naming convention used in official Arduino examples, please rename the sketch folder and file ServoStop.

Please add a comment containing a short description of the purpose of the sketch and the required circuit. You can check the library's other examples to see the format.

examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
#define trigPin 4
#define echoPin 2

Servo myservo; // create servo object to control a servo
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Servo myservo; // create servo object to control a servo
Servo myservo; // create Servo object to control a servo

Use correct class name.

Copy link
Author

Choose a reason for hiding this comment

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

PR Updated

Copy link
Contributor

Choose a reason for hiding this comment

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

This was not done.

examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/Servo-Stop/Servo-Stop.ino Outdated Show resolved Hide resolved
examples/ServoStop/ServoStop.ino Outdated Show resolved Hide resolved
examples/ServoStop/ServoStop.ino Outdated Show resolved Hide resolved
examples/ServoStop/ServoStop.ino Outdated Show resolved Hide resolved
@CLAassistant
Copy link

CLAassistant commented Apr 9, 2021

CLA assistant check
All committers have signed the CLA.

@per1234 per1234 dismissed their stale review May 15, 2021 23:28

All significant requested changes have been resolved. Thanks!

@per1234 per1234 added type: enhancement Proposed improvement topic: documentation Related to documentation for the project labels Jun 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: documentation Related to documentation for the project type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants