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

Divide repair_robot to two methods and change the logic of calling them. #258

Merged
merged 2 commits into from Jun 12, 2019

Conversation

Projects
None yet
2 participants
@befeleme
Copy link
Contributor

commented Jun 8, 2019

Jak jsme se bavili na srazu, rozdělila jsem funkčnost repair_robot.
Nově: v apply_tile_effects je volaná nová metoda set_new_start, která robotům stojícím na políčkách repair změní startovní koordináty.
Zato do set_robots_for_new_turn byla přenesená metoda repair_robot, která opravuje pouze roboty stojící na políčkách repair v momentu zavolání, tedy vždy na konci tahu. Tím se políčka přestala zajímat o argumenty register, registers.
Druhým commitem jsem předělala docstring v interface'u, na jehož znění mě upozornil @encukou v review.

befeleme added some commits Jun 8, 2019

Split repair_robot to repair_robot and set_new_start as there are two…
… funcionalities. Change the backend logic of calling them.

@ivet1987 ivet1987 merged commit b429964 into PyLadiesCZ:master Jun 12, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@befeleme befeleme deleted the befeleme:repair-change branch Jun 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.