Skip to content

Added robot waggin#31

Open
satsab02 wants to merge 12 commits into
masterfrom
playSoundFunc
Open

Added robot waggin#31
satsab02 wants to merge 12 commits into
masterfrom
playSoundFunc

Conversation

@satsab02
Copy link
Copy Markdown
Collaborator

No description provided.

@satsab02 satsab02 requested a review from fubblea March 27, 2025 16:53
Comment thread src/mie443_contest3/include/contest3/callbacks.h Outdated
Comment thread src/mie443_contest3/include/contest3/state.h Outdated
Comment thread src/mie443_contest3/include/contest3/state.h Outdated
Copy link
Copy Markdown
Owner

@fubblea fubblea left a comment

Choose a reason for hiding this comment

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

Test

Copy link
Copy Markdown
Owner

@fubblea fubblea left a comment

Choose a reason for hiding this comment

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

NO

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Stick this in state.h.

No need for it's own file

Comment on lines +104 to +114
if (doTurn(45, stateHist.back().yaw, true)) {
ROS_INFO("Turning one way");
return;
}
if (doTurn(-90, stateHist.back().yaw, true)) {
ROS_INFO("Turning the other way");
return;
}
if (doTurn(45, stateHist.back().yaw, true)) {
ROS_INFO("Looking straight");
return;
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

These have to be nested

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This also has to be interruptable

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.

4 participants