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

use built-in portal functionality for Hackman scenario #1651

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

kostmo
Copy link
Member

@kostmo kostmo commented Nov 22, 2023

Previously, the logic to wrap horizontally was hand-coded, both for the player (using a dedicated monitor robot) and for ghosts (who, as system robots, implemented the teleportation themselves).

Now that we can use built-in portal functionality (#1356), no custom logic is needed to handle the teleportation. A (much-simplified) monitor robot still observes the base robot to award the hidden "World wrap" goal.

Demo

scripts/play.sh -i data/scenarios/Challenges/hackman.yaml --autoplay --speed 6

@kostmo kostmo added the G-Scenarios An issue having to do with scenario design, the way scenarios are described and loaded, etc. label Nov 22, 2023
@kostmo kostmo marked this pull request as ready for review November 22, 2023 06:36
@kostmo kostmo requested a review from byorgey November 22, 2023 06:36
@kostmo kostmo added the merge me Trigger the merge process of the Pull request. label Nov 22, 2023
@mergify mergify bot merged commit 40f26df into main Nov 22, 2023
9 checks passed
@mergify mergify bot deleted the scenario/hackman-portals branch November 22, 2023 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
G-Scenarios An issue having to do with scenario design, the way scenarios are described and loaded, etc. merge me Trigger the merge process of the Pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants