search

strands-project / strands_morse

A MORSE-based simulation for the STRANDS project
4 stars 28 forks source link

Update simulated laser pose to match real robot. #147

Closed cburbridge closed 8 years ago

cburbridge commented 8 years ago

Closes https://github.com/strands-project/strands_morse/issues/143.

cburbridge commented 8 years ago

Is there any way to have Jenkins run the navigation tests with this PR?

Jailander commented 8 years ago

To run them at jenkins I guess we would need to create a PR in strands_navigation after this has been merged (right @marc-hanheide ?) but you can run the tests on your workspace following the instructions here: https://github.com/strands-project/strands_navigation/tree/indigo-devel/topological_navigation/tests

marc-hanheide commented 8 years ago

Yes, please run these separately. We can't have a full test in this repo

marc-hanheide commented 8 years ago

@Jailander can you comment? Did you try? I think we should merge this, as it certainly is not making things fail, but if it improves anything we need to still see.

Jailander commented 8 years ago

Yes I tried it didn't break as you said not sure it helps merge if you want