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

Find robots #339

Merged
merged 1 commit into from
Jun 4, 2022
Merged

Find robots #339

merged 1 commit into from
Jun 4, 2022

Conversation

xsebek
Copy link
Member

@xsebek xsebek commented May 22, 2022

  • add robotNamed : string -> cmd robot
  • add chess horse challenge 🏇

In challenges, it's impossible to know beforehand what the robot ID will be.
Seems like we should be able to find it by its name though.

@xsebek xsebek requested a review from byorgey May 22, 2022 09:39
@xsebek
Copy link
Member Author

xsebek commented May 22, 2022

The challenge is a bit hard* but not impossible:
Screenshot from 2022-05-22 11-42-54

move; move; move; turn right; move; turn left; move;

Screenshot from 2022-05-22 11-43-07

I like that we can customize all this in current challenges. 👍
Being able to query the position of an arbitrary robot makes it even sweeter.


*) Unless you turn east and cheese it of course 😀

Copy link
Member

@byorgey byorgey left a comment

Choose a reason for hiding this comment

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

Makes sense, I like it.

Hmm, maybe there should be a "compass" device which you need to use directions like east, west etc. Then we could prevent cheesing this challenge by not starting the robot with a compass...

data/challenges/chess_horse.yaml Outdated Show resolved Hide resolved
This was referenced May 28, 2022
@xsebek xsebek marked this pull request as ready for review June 3, 2022 11:36
* add robotNamed : string -> cmd robot
* add chess knight challenge

In challenges, it's impossible to access the robot
by its assigned number. We do however name it.
Seems reasonable to add a way to get robot by its name.
@xsebek xsebek merged commit 161f510 into main Jun 4, 2022
@xsebek xsebek deleted the find-robot branch June 4, 2022 22:29
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