Skip to content

Conversation

@jamsrahimi
Copy link

Dear Sir,
I did bring changes to the robot, please grant me a pull request.

Copy link
Owner

@remarcmij remarcmij left a comment

Choose a reason for hiding this comment

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

Hey Jams, excellent work!

render();

})();
turn('down');
Copy link
Owner

Choose a reason for hiding this comment

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

I guess this was a test with 'down'? When you submit a pull request make sure you take out all your debug code.

Copy link
Author

@jamsrahimi jamsrahimi Jan 12, 2018

Choose a reason for hiding this comment

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

Yes, I tried to decode the concept behind the directions, it did not make sense. I did not notice the debug code was there. Wow I just notice what the direction mean.

Copy link
Owner

Choose a reason for hiding this comment

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

Yes, the right and left are relative turn directions. up, down, left and right are the absolute (compass) directions for the robot. Is this now your current understanding?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, that is my current understanding now. But the direction cannot change from → to ←? It can only turn to either left or right.

Copy link
Owner

Choose a reason for hiding this comment

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

Turn left or right twice in a row?

See the diagram in the README. You did see that, did you?

Copy link
Author

Choose a reason for hiding this comment

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

Seeing the arrow keys in keyboard, as a 'confirmation bias' I had assumed to see a down or up turn. Now I understand it completely after seeing the README just now. I had seen it on your youtube recording.

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