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

[Feature]: Behavior tree cleanup code in preparation for including intermediate map #589

Open
Johannes1098 opened this issue Dec 20, 2024 · 0 comments
Assignees
Labels

Comments

@Johannes1098
Copy link
Collaborator

Johannes1098 commented Dec 20, 2024

Feature Description

The code is sometimes inconsistent, some code fragments are from the project 3 years ago and is not necessary anymore. Also some function names are inconsistent and also incomprehensible (e.g. convert_to_ms. is it milliseconds or meter/second. nobody knows at first sight).

With the more readable code we can implement the intermediate map faster in the first january weeks as we know where and what to add.

Definition of Done

  • The code should me more consistent.
  • Old legacy code that isn't is removed.
  • Add comments where needed.
  • Function name

Testability

Test if car / simulation and especially the behavior tree acts like before. The behavior tree can be looked up with the rqt_py_trees RQT Plugin (check if same behaviors still exists)

Dependencies

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🏗 In progress
Development

No branches or pull requests

1 participant