Skip to content

added changes to support multi levels based on difficulty - #785

Open
anishk85 wants to merge 1 commit into
JdeRobot:humble-develfrom
anishk85:drone_cat_mouse_levels
Open

added changes to support multi levels based on difficulty#785
anishk85 wants to merge 1 commit into
JdeRobot:humble-develfrom
anishk85:drone_cat_mouse_levels

Conversation

@anishk85

Copy link
Copy Markdown
Collaborator

…a course parameter to support multi levels in drone cat mouse.

@javizqh

javizqh commented Jul 30, 2026

Copy link
Copy Markdown
Member

Please state the purpose of adding a parameter that is not used anywhere.

@anishk85

anishk85 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator Author

it was done to support different levels of chase for mouse drone.

suppose this is set Hard
the HAL api when calls get parameters function it receives course hard and based on that mouse drone follows a predefined path.

for cat this will be empty.

and for other excercise not using this they can simply pass it as it is a parameter.

@anishk85

anishk85 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator Author

I have one more way of doing this via introducing three launchers
drone_cat_mouse_easy.launch.py course = "easy"
drone_cat_mouse_medium.launch.py course = "medium"
drone_cat_mouse_hard.launch.py course = "hard"

that requires change in db passing extra config

it will prevent quadrotor.launch.py to pass that course parameter

@anishk85

Copy link
Copy Markdown
Collaborator Author

this pr is for discussion of this only which way should i go ?

@pariaspe told me to discuss this with you.

for me both approaches seems fine as by introducing new parameter we are opening a path for any future excersice as well that might need some parameter or something.

do let me know your opinion as well.

@javizqh

javizqh commented Jul 31, 2026

Copy link
Copy Markdown
Member

If this is meant to combine with other changes, please state it at the beginning, adding the link to the remaining PR. I cannot know what you try to do if you only open a PR for a part of the changes.

@anishk85

anishk85 commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator Author

If this is meant to combine with other changes, please state it at the beginning, adding the link to the remaining PR. I cannot know what you try to do if you only open a PR for a part of the changes.

sure sure in coming days i will push the pr on ra also using these changes once i finalize with algorithms that we will be using.

for now if you want i can push a minimal verison of that.

@anishk85

anishk85 commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator Author

but i need an answer about the approach is it okay to go with this pr or the other approach which i explained earlier.
because changes in ra(drone cat mouse related) are dependent on this pr,

@javizqh

javizqh commented Aug 2, 2026

Copy link
Copy Markdown
Member

I cannot test this without the others changes

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants