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

Is train to advanced to be a demo robot? #77

Closed
dbakewel opened this issue May 31, 2019 · 5 comments
Closed

Is train to advanced to be a demo robot? #77

dbakewel opened this issue May 31, 2019 · 5 comments
Labels
enhancement New feature or request
Milestone

Comments

@dbakewel
Copy link
Owner

@mcstoer suggested that the train.py demo robot moment is to advanced for a demo robot. Some players simply use train.py movement without much modification.

I would like to see a discussion in the comments. Should train.py be removed from demo robots and leave it up to players to figure movement out on their own? What kind of demo robot should replace train.py?

@dbakewel dbakewel added the question Further information is requested label May 31, 2019
@dbakewel dbakewel pinned this issue May 31, 2019
@MCSnapTurtle
Copy link
Contributor

@mcstoer has a valid point. Compared to the other starting bots, train is the only moving bot that can avoid walls. The rest either ram into walls or don't move at all. Wall avoidance is something quite advanced. Yes, it is probably a good idea to remove train, because it's movement alone is superior to anything I have made (I have a wall avoiding, random direction moving bot).

You could replace train with a variant that does not turn when it is close to the wall. Instead, once it hits the wall, it changes direction. So basically, it moves like train, except it hugs the walls. So it still keeps track of where it is, and which wall it is beside, but only uses that information once it rams into a wall. This is definitely worse, but still keeps the "personality" of train, so I think it is a good replacement.

I could work on the "wall banging" train if the idea is fine.

@dbakewel
Copy link
Owner Author

I like the wall banging idea. @MCSnapTurtle I think you have enough to work on with #47. Let's wait for others feedback on this.

@MCSnapTurtle
Copy link
Contributor

@dbakewel I would love to do the train changes before doing #47, because the train changes should be pretty quick to implement.

@MCSnapTurtle
Copy link
Contributor

@mcstoer and I have thought of a new demo bot. It find the closest enemy and moves in the opposite direction from it. I'll be working on this.

@JamesDarby345
Copy link
Contributor

The train movement is rather good, it is what I have used for my bot but I just made it faster and turn sooner. It probably should be replaced or simplified.

@dbakewel dbakewel unpinned this issue Jun 4, 2019
@dbakewel dbakewel added enhancement New feature or request and removed question Further information is requested labels Jun 4, 2019
@dbakewel dbakewel added this to the 2.0 milestone Jun 4, 2019
@dbakewel dbakewel closed this as completed Jun 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants