Levi Armstrong

> Overall I like the PR. I have a concern about performance. Currently this functions is used a lot withing motion planning which can be called thousands of times within...

Also think it would be good to add google benchmark to show the performance between the two is unchanged. I recently did this for the type erasure due to the...

@marrts Just as a precaution do you mind creating a simple google benchmark for this?

Thank you for the information. We have plans to move the documentation to its own repository which will happen in the near future. What would be a good name for...

The documentation has been moved but not sure if it resolves the issue.

I see, so I need to also move the doxygen and benchmark data to the docs repo and then purge the branch. Is that correct?

This issue is the two places where the following were added to solve windows ros build issues. I am going to wrap them with windows compiler def to solve the...

The default resource locators support this so it makes sense that they would support it also.

Maybe we create a base class called ResourceExporter with a virtual method given a file name generates a URI. Then all of the write methods in tesseract_urdf take this instead...

@mpowelson What do you think we should do with this PR?