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.

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

Simulate robot battery usage and charging #235

Merged
merged 8 commits into from
Aug 7, 2024
Merged

Simulate robot battery usage and charging #235

merged 8 commits into from
Aug 7, 2024

Conversation

sea-bass
Copy link
Owner

@sea-bass sea-bass commented Aug 7, 2024

This PR adds battery level to robots, as well as the option to simulate battery usage per action.

As a side effect, this PR also correctly applies action execution options from manually interacting with the GUI, which was not previously available.

image

Closes #230

Copy link

github-actions bot commented Aug 7, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
core
   __init__.py90100% 
   dynamics.py430100% 
   gazebo.py136894%55, 124, 230, 244–248
   hallway.py92891%192, 234–237, 261–263
   locations.py1312183%69, 71, 88, 103–104, 106, 117–118, 120–121, 138, 196, 211, 236, 246, 291, 302–303, 305–306, 335
   objects.py69494%78, 106, 137, 197
   robot.py4596685%259–260, 272, 275, 279–280, 285–286, 334–335, 359, 425–428, 432–435, 480, 482, 524–525, 527–528, 545–547, 563–565, 632–634, 659–661, 712–714, 783–785, 795, 839–841, 851, 866, 872–873, 875, 877, 880–882, 895, 903, 909, 915, 921, 933, 975, 993, 996, 1001
   room.py48197%113
   world.py73518674%147–148, 184–185, 189, 242–243, 245–246, 274–275, 298–300, 337–339, 351–353, 382–384, 409–411, 464, 478–479, 481–482, 497–499, 515–519, 521–523, 525–526, 529, 533–537, 540–542, 545–556, 572–573, 591–593, 607–609, 645–647, 659–661, 683–685, 710–712, 760–761, 764, 780, 810–811, 822, 825, 833, 849–853, 855–857, 860–861, 864–867, 869, 872, 874–876, 878–880, 882, 897, 904, 915–919, 954–955, 959–960, 978–979, 985–986, 988–989, 991, 993–994, 996, 998–999, 1009–1010, 1013, 1015, 1029, 1031, 1038, 1085, 1116–1117, 1128, 1139–1141, 1143–1146, 1148, 1165–1166, 1170–1171, 1206–1207, 1209, 1226–1227, 1258, 1272–1275, 1287–1290, 1304, 1315–1316, 1318, 1362–1363, 1415, 1495–1496
   yaml_utils.py1151190%65, 69, 156–158, 162–163, 183–184, 203–204
gui
   __init__.py20100% 
   main.py1903084%22–24, 45, 77–78, 196, 208–212, 219–222, 224, 227–233, 237, 242–243, 247–248, 259
   world_canvas.py2845580%44–45, 47–51, 54, 62–63, 171–174, 181, 183, 240–242, 289, 326, 360–361, 384–385, 387, 389–391, 394–397, 399, 403–409, 412, 415–416, 431–433, 436, 455, 498, 519, 523, 546, 563, 577
manipulation
   __init__.py10100% 
   grasping.py2366174%102, 110–112, 114–122, 124, 132, 462, 464–466, 470–472, 566–568, 571–575, 583, 586–588, 593–594, 600–604, 607–614, 617–619, 628, 630, 632–638
navigation
   __init__.py30100% 
   a_star.py781087%67–72, 74–76, 165
   execution.py90693%73–75, 80–82
   occupancy_grid.py1391489%44–47, 49–53, 56, 192, 226, 234, 238
   path_planner.py26676%33, 36, 38, 42, 89, 94
   planner_base.py591771%29–35, 51, 57–59, 142–147
   prm.py72297%61–62
   rrt.py1740100% 
   world_graph.py70395%118, 121, 145
planning
   __init__.py00100% 
   actions.py1100100% 
planning/pddlstream
   __init__.py7185%13
   default_mappings.py90100% 
   planner.py40880%119–126
   primitives.py57296%68, 208
   utils.py85396%66, 99–100
utils
   __init__.py00100% 
   general.py33487%23, 45, 51–52
   knowledge.py1542285%86, 117, 156–161, 163–164, 167–168, 170–173, 189, 204, 208, 250, 266, 329
   motion.py650100% 
   polygon.py1110100% 
   pose.py890100% 
   search_graph.py93396%240–242
   trajectory.py67198%165
TOTAL418155386% 

Tests Skipped Failures Errors Time
196 0 💤 0 ❌ 0 🔥 3m 10s ⏱️

@sea-bass sea-bass marked this pull request as ready for review August 7, 2024 04:08
@sea-bass sea-bass merged commit d581cdf into main Aug 7, 2024
10 of 13 checks passed
@sea-bass sea-bass deleted the battery-sim branch August 7, 2024 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add battery simulation capabilities
1 participant