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

Add shipghost team solution #24

Merged
merged 1 commit into from
Dec 2, 2020
Merged

Add shipghost team solution #24

merged 1 commit into from
Dec 2, 2020

Conversation

mabelzhang
Copy link
Collaborator

@mabelzhang mabelzhang commented Nov 24, 2020

Add shipghost solution discussed in #19 thanks to @crvogt, so that VORC has more than one team to test the multiscripts.

To test the multiscripts:

(Re)move the generated/ directory, to start from scratch (so that the team score script doesn’t find extra tasks from previous runs and think a task score is missing):

mv generated generated_bak

This should prep both ghostship and shipghost teams:

./multi_scripts/prepare_all_teams.bash

Prepare just one task so the runs don’t take forever. I picked perception, the quickest to run.
Optionally, you could pick a slower task and manually change the <running_state_duration> to a shorter duration (default 300 seconds) in generated/task_generated/*/worlds/*.world files.

./prepare_task_trials.bash perception

This should run both teams on all trials of the perception task:

./multi_scripts/run_all_teams_all_tasks.bash -n

This is… quite entertaining, especially the third trial. The boat eventually got flipped by the wind and disappeared in the mist.

To test the new solution works:

Check that the boat moves backwards in a task that doesn’t lock the boat (i.e. not perception):

./prepare_task_trials.bash wayfinding
# Optionally, shorten duration in generated/task_generated/wayfinding/worlds/wayfinding0.world to <running_state_duration>60
./run_trial.bash -n shipghost wayfinding 0

@mabelzhang mabelzhang merged commit b0631d8 into vorc Dec 2, 2020
@mabelzhang mabelzhang deleted the add_shipghost branch December 2, 2020 21:19
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