Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[improvement] rails in base do u-turn #731

Merged
merged 2 commits into from Mar 4, 2012

Conversation

Projects
None yet
3 participants
Contributor

Elosanda commented Mar 4, 2012

also changed levels to reflect this

see issue #724

Contributor

Elosanda commented Mar 4, 2012

you can build a closed circle where the RailBots float without collisions. thats not possible with the old rail connection

Contributor

danielduner commented Mar 4, 2012

The big advantage that I see is that it makes it possible (and encourages) a big loop for the rail droids. Instead of the insane back-and-forth bouncing. If we just fix #608 as well, the droids will almost be functional.

Collaborator

master-lincoln commented Mar 4, 2012

convinced...

Contributor

danielduner commented Mar 4, 2012

As the response in #724 was very positive as well, I'll just go ahead and merge this. Blame me if anything goes wrong!

danielduner added a commit that referenced this pull request Mar 4, 2012

Merge pull request #731 from Elosanda/improv/RailsInBase
[improvement] rails in base do u-turn

@danielduner danielduner merged commit c3ad102 into Maescool:develop Mar 4, 2012

Collaborator

master-lincoln commented Mar 4, 2012

Again daniel?
Remember the last time you said that... ^^

Contributor

danielduner commented Mar 4, 2012

The one merging a commit is responsible. I probably should test the code first, but I'm too lazy. So instead, I just jump in head first and wait for the shit to hit the fan!

Contributor

Elosanda commented Mar 4, 2012

we must have punishment for such reckless behavior ;)
i will also look into #608

Collaborator

master-lincoln commented Mar 4, 2012

getting out his umbrella to shield the shit ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment