Improvements during debug session at Inria testbed

[geonnave]

No description provided.

[codecov]

Codecov Report

❌ Patch coverage is 45.83333% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.60%. Comparing base (05e6313) to head (bc1f9af).
⚠️ Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
dotbot/examples/work_and_charge/work_and_charge.py	28.57%	5 Missing ⚠️
...tbot/examples/charging_station/charging_station.py	42.85%	4 Missing ⚠️
...mum_naming_game/minimum_naming_game_with_motion.py	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #216      +/-   ##
==========================================
- Coverage   68.84%   68.60%   -0.24%     
==========================================
  Files          40       40              
  Lines        3739     3752      +13     
==========================================
  Hits         2574     2574              
- Misses       1165     1178      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[aabadie]

How about keeping this module empty and modify the import in the corresponding test module? I would find it more convenient as it's only a one liner, even if you have to do import from dotbot.examples.charging_station.charging_station import ....

[WilliamTakeshi]

fixed on fcf2c16 (this PR)

[aabadie]

Can you explain the use case of this example?

[geonnave]

Yes: when we run any of the examples in a real robot, then when (1) when we stop the example with ctrl+c or (2) the example script crashes, the robots just keep running like crazy. This stop commands make them stop, without having to call swamit stop (and then swarmit start again). Technically we could just use swarmit stop, but right now it takes like 5 seconds or more, which is annoying from a development experience PoV.

[aabadie]

but could it be done in the example script directly?
in a "catch KeyboardInterrup" block, send the empty waypoint and return

[geonnave]

Makes sense. @WilliamTakeshi can you take a look?

[WilliamTakeshi]

fixed on 19db2ec (this PR)