Adding node blacklisting to executive

strands-project / strands_executive

Executive control code for STRANDS robots.

10 stars 19 forks source link

Adding node blacklisting to executive #312

Closed hawesie closed 5 years ago

hawesie commented 5 years ago

This is a work in progress and should not be merged

Depends on #311 and https://github.com/strands-project/strands_navigation/pull/369 so if testing, make sure you have those locally (this PR actually subsumes #311)

To test, edit task_executor/scripts/example_blacklist_provider.py to set some nodes as blacklisted, then add tasks to the task_executor as normal. Expected behaviour is that these nodes are not visited in the policy, but that's not currently the case.

hawesie commented 5 years ago

@francescodelduchetto this is ready for testing. It should do what you want!

francescodelduchetto commented 5 years ago

@hawesie I did some testing and it looks all right! Thanks

hawesie commented 5 years ago

If you can poke @Jailander into checking https://github.com/strands-project/strands_navigation/pull/369 then we can merge both.