Vince Reda
Vince Reda
Nice catch! I was facing some issues with this. Originally the minimum_travel_heading parameter existed but it was never used, which was misleading. Also, I think a separate parameter should not...
Woah, this suddenly became more serious than I expected :D I still have to write the tests and make this work with my jazzy stack, so it might take some...
@SteveMacenski I started writing the tests and I have a few questions. The dummy BT node used for tests now returns `IDLE` after reset. Shouldn't it always return the status...
I found the bug which didn't let me set `current_child_idx` input_port: #5295
I've implemented the tests using xml strings as in the action nodes, and DummyNodes. I've had to be creative, because none of the existing tests needed this level of complexity....
@SteveMacenski can you take a look at this?
**WARNING: don't accept this yet, still waiting for submodule changes** I've finally implemented the requested changes, but it turns out to inherit from the `SequenceNode`, more changes are required (namely...
Okay! Then let's wait for the BT.CPP release, and then I'm gonna open a follow up. Until then, this PR should be ready to merge :D I removed the parts...
My main source of confusion is that I don't know what the difference is between condition and action nodes. If I understand correctly, a condition node returns SUCCESS or FAILURE...
I think it's only used in tests, so it doesn't affect functional code. It was initially blocking #5247, but then I went with another solution, using blackboard variables. It's just...