Skip to content

Striker Subtree#2436

Open
pejotejo wants to merge 19 commits intoHULKs:mainfrom
pejotejo:striker_tree
Open

Striker Subtree#2436
pejotejo wants to merge 19 commits intoHULKs:mainfrom
pejotejo:striker_tree

Conversation

@pejotejo
Copy link
Copy Markdown
Contributor

@pejotejo pejotejo commented Apr 18, 2026

depends on #2435

Why? What?

Adds a more complex striker tree with switching between walking and kicking and switching beween the different kicking policies. The current tree should prevent instant switches between the different policies. This should hopefully stabilize its motions. Also its adds an interceptmotion.

ToDo / Known Issues

think about how we want to split up the kicking motion with kickingpower and target

Ideas for Next Iterations (Not This PR)

handle headmotion seperately (currently working on it)

How to Test

Play with the robot and look at the behavior tree.

@github-project-automation github-project-automation Bot moved this to Request for Review in Development Apr 18, 2026
@pejotejo pejotejo changed the title Striker tree Striker Subtree Apr 18, 2026
@pejotejo pejotejo enabled auto-merge April 18, 2026 13:02
@pejotejo pejotejo disabled auto-merge April 18, 2026 13:02
@pejotejo pejotejo moved this from Request for Review to In Progress in Development Apr 18, 2026
@pejotejo pejotejo force-pushed the striker_tree branch 2 times, most recently from 1c81ce6 to 613fa41 Compare April 19, 2026 16:02
@pejotejo pejotejo enabled auto-merge April 19, 2026 16:02
@pejotejo pejotejo moved this from In Progress to Request for Review in Development Apr 21, 2026
@paudar paudar self-assigned this Apr 22, 2026
Comment thread crates/types/src/motion_type.rs Outdated
}

impl MotionType {
pub fn is_stable(&self) -> bool {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused function?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was from the old behavior, is now gone

Comment thread crates/world_state/src/behavior/tree.rs Outdated
selection!(
sequence!(
condition!(is_last_motion_type, MotionType::Walk),
action!(set_is_alternative),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In my view, the set_is_alternative function is not relevant

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is removed

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

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

2 participants