Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Windows fixes for statistics tutorial #533

Merged
merged 3 commits into from
Sep 26, 2024
Merged

Windows fixes for statistics tutorial #533

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4c963a2
Windows fixes for statistics tutorial
j-rivero Sep 12, 2024
cf8d745
Do not use cmake --config for single configuration generators
j-rivero Sep 13, 2024
490975b
Merge branch 'gz-transport14' into jrivero/t14_statistic_win
j-rivero Sep 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 19 additions & 5 deletions tutorials/23_topic_statistics.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ for both publishers and subscribers. Setting `GZ_TRANSPORT_TOPIC_STATISTICS` to
Additionally, a node on the subscriber side of a pub/sub relationship must
call `EnableStats`. For example:

```
```c++
if (!node.EnableStats(topic, true))
{
std::cout << "Unable to enable stats\n";
Expand All @@ -54,7 +54,7 @@ It is possible to change the statistics output topic from `/statistics` to
one of your choosing by specifying a topic name when enabling topic
statistics. For example:

```
```c++
if (!node.EnableStats(topic, true, "/my_stats"))
{
std::cout << "Unable to enable stats\n";
Expand All @@ -64,7 +64,7 @@ if (!node.EnableStats(topic, true, "/my_stats"))
You can also change the statistics publication rate from 1Hz by specifying
the new Hz rate after the statistic's topic name:

```
```c++
if (!node.EnableStats(topic, true, "/my_stats", 100))
{
std::cout << "Unable to enable stats\n";
Expand All @@ -73,9 +73,23 @@ if (!node.EnableStats(topic, true, "/my_stats", 100))

### Example

If you have the Gazebo Transport sources with the example programs built,
then you can test topic statistics by following these steps.
For running the example, build the binaries in the example directory:

```
git clone https://github.com/gazebosim/gz-transport -b gz-transport14
cd gz-transport/example
cmake -S . -B build
cmake --build build --config release --parallel
azeey marked this conversation as resolved.
Show resolved Hide resolved
```

#### Executing on Linux or Mac

1. Terminal 1: `GZ_TRANSPORT_TOPIC_STATISTICS=1 ./example/build/publisher`
1. Terminal 2: `GZ_TRANSPORT_TOPIC_STATISTICS=1 ./example/build/subscriber_stats`
1. Terminal 3: `GZ_TRANSPORT_TOPIC_STATISTICS=1 gz topic -et /statistics`

#### Executing on Windows

1. Terminal 1: `set "GZ_TRANSPORT_TOPIC_STATISTICS=1" && example\build\release\publisher.exe`
1. Terminal 2: `set "GZ_TRANSPORT_TOPIC_STATISTICS=1" && example\build\subscriber_stats.exe`
1. Terminal 3: `set "GZ_TRANSPORT_TOPIC_STATISTICS=1" && gz topic -et /statistics`
Loading