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

Update Drone_sensor_library.ino - WSchuler #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

weschuler
Copy link
Contributor

Addition of comments and questions to the code. Marked by WSchuler

Addition of comments and questions to the code. Marked by  WSchuler
Copy link
Collaborator

@zbkitz zbkitz left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the comments on the messages and publishers.
In lines 161 and 231, each case is a different "mode" for the system; each case is already labeled with the name of the corresponding mode in the line above each case statement (e.g. lines 162 and 232). Are you asking me to add a summary of the function of each mode?
I'm going to hold off on merging until you've had a chance to review the rest of the code.

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

Successfully merging this pull request may close these issues.

2 participants