-
Notifications
You must be signed in to change notification settings - Fork 3
"Updated offensive behaviors, and… #3
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
Conversation
… modified KickDecisionMaker to use BhvStarterKickPlanner."
src/behaviors/bhv_starter_pass.py
Outdated
| from service_pb2 import * | ||
|
|
||
|
|
||
| class Pass(IBehavior): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class name should be BhvStarterPass
src/behaviors/bhv_starter_dribble.py
Outdated
| from service_pb2 import * | ||
|
|
||
|
|
||
| class Dribble(IBehavior): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BhvStarterDribble
| from pyrusgeom import vector_2d | ||
| from service_pb2 import * | ||
|
|
||
| class ClearBall(IBehavior): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bhv...
| agent.logger.debug("--- WithBallDecisionMaker ---") | ||
| from src.sample_player_agent import SamplePlayerAgent # Local import to avoid circular import | ||
| assert isinstance(agent, SamplePlayerAgent) | ||
| #self.bhv_kick_planner.execute(agent) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we do not need this line
| def __init__(self): | ||
| self.bhv_kick_planner = BhvKickPlanner() | ||
| # self.bhv_kick_planner = BhvStarterKickPlanner() | ||
| self.bhv_kick_planner = BhvStarterKickPlanner() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for default we should not use Starter classes, but we can have a variable in sample player agent, for example starter=true/false and we can choose planner or starter planner based on that variable
SoroushMazloum
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed these issues in my new commit
…setplay files: go_to_placed_ball, intention_wait_after_setplay, prepare_setplay_kick, setplay_freekick. add starter_strategy
This reverts commit 228f389. (Erfan should add this part)
…t_play_decision_maker.py. Added two new methods to tools.py: BallInertiaFinalPoint and OpponentGoalie. Also added some setplay files`
- Designed penalty kick behavior in `BhvStarterPenalty`: - Added kicker setup, readiness, and execution methods. - Implemented goalie actions: setup, waiting, and responses. - Enhanced decision-making for `dribbling` and `shooting`. - Improved goalie positioning based on ball dynamics. - Included support for various penalty game modes.
… modified KickDecisionMaker to use BhvStarterKickPlanner."