gazebosim / gazebo_test_cases

4 stars 1 forks source link

gz-transport: Topic Statistics #1566

Open azeey-test opened 2 weeks ago

azeey-test commented 2 weeks ago

Configuration

Links

Process

Step 1 - Instructions work

Step 2 - Images (if there are any) match the result

peci1 commented 1 week ago

I'll test this over the weekend on a real Win 10 laptop.

j-rivero commented 5 days ago

Using a from source installation on Windows, the example is not trivial to build on Windows without the CMakeLists.txt. Need to patch the documentation to use the examples directory CMakeLists.txt.

We could benefit from moving the building commands to use platform independent cmake equivalents: i.e: cmake .. -> cmake -S . -B build/ -D CMAKE_BUILD_TYPE=Release && make -> cmake --build build/.

One liners need to be changed: set "GZ_TRANSPORT_TOPIC_STATISTICS=1" && example\build\Release\publisher.exe

j-rivero commented 5 days ago

Functionality works.

j-rivero commented 5 days ago

PR https://github.com/gazebosim/gz-transport/pull/533