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

halt command #1256

Merged
merged 10 commits into from May 19, 2023
Merged

halt command #1256

merged 10 commits into from May 19, 2023

Conversation

byorgey
Copy link
Member

@byorgey byorgey commented May 13, 2023

Closes #392 . Adds a command halt : actor -> cmd unit which halts the given robot if it is within a distance of 1 (no distance limit for system robots or in creative mode). halt self works too. Privileged robots (i.e. system robots, or when in creative mode) can halt any other robot. Unprivileged robots cannot halt system robots.

Comment on lines +1258 to +1260
- A device to solve the halting problem. When asked if a
particular robot program will halt, it always answers YES.
And it is always correct... or else!
Copy link
Member

Choose a reason for hiding this comment

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

This is great.

@byorgey
Copy link
Member Author

byorgey commented May 13, 2023

Oh, I forgot to add some tests! I will do that before merging.

@byorgey byorgey added the merge me Trigger the merge process of the Pull request. label May 18, 2023
@mergify mergify bot merged commit 2487737 into main May 19, 2023
10 checks passed
@mergify mergify bot deleted the feature/halt branch May 19, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Trigger the merge process of the Pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

halt command to cancel a robot's running program
2 participants