Davide Faconti

Results 443 comments of Davide Faconti

Is this change tested on Windows? We don't want to break it there

Unit tests are failing. Please remember to run the tests before submitting a PR :)

Groot can't be used like that, so, from my side, the answer is "no"

1. Is this change back-compatible with the previous JSON format? 2. Can we opt-out if people don't want to use this format?

The correct solution, in my opinion, is to presere the order of the ports as it was described in **providedPorts**. Unfortunately, this means that we need to apply a change...

I need a way to reproduce this. Is there any dummy server and client that can be used to reproduce the error? It seems to me like a potential error...

you are welcome to submit a PR to address this.

More likely the problem is in the ROS subscriber node, not in Parallel. I will investigate.