Modular graphnav#6
Closed
heuristicus wants to merge 81 commits intomainfrom
Closed
Conversation
* use pytest instead * add pytest dependency * fail test on purpose * remove failing test
…pot_ros2 (rai-opensource#7) Co-authored-by: Andrew Messing <129519955+amessing-bdai@users.noreply.github.com>
* simple led brightness control, only able to set all leds to same value * Add power control, but unclear if it is actually possible to set power for aux and external mic * most basic functional image stream publisher with webrtc * add compositor to handle IR and webrtc stream selection with services Add timestamp for the webrtc images Add compressed version of the webrtc image stream * Add health wrapper, move body of robotToLocalTime out of spot wrapper object robotToLocalTime now takes the timestamp and a robot object, which allows it to be used by the spot cam wrapper as well. * add handler and wrapper for audio commands * update webrtc_client to 3.2 version * add stream quality wrapper and ros handler * initial implementation of ptz wrapper and handler, can list ptzs * ptz handler publishes position and velocity of ptzs, can set position and velocity * add egg info to gitignore
* change the try-claim decorator to an object * claim decorator takes functions it needs, no longer relies on power being decorated Previously the claim decorator assumed that the power_on function was decorated with the same decorator. If power on was requested, the power on function would do the claiming, and then power on the system. That is kind of confusing, so now the decorator will call claim first, and then optionally power on. Also takes the power on and claim functions, rather than calling the ones on the wrapper itself. * formatting * move decorator object to helper file, rename and make functions better, decorate docking functions * formatting * remove mistakenly merged robot command data and state * decorate arm commands * Fixed bugs with decorated functions * black --------- Co-authored-by: Andrew Messing <129519955+amessing-bdai@users.noreply.github.com> Co-authored-by: Andrew Messing <amessing@theaiinstitute.com>
bhung-bdai
reviewed
Aug 3, 2023
| ko_tform_body=current_odom_tform_body, | ||
| ) | ||
|
|
||
| def set_initial_localization_waypoint(self, *args): |
bhung-bdai
reviewed
Aug 3, 2023
| f"Got robot state in kinematic odometry frame: \n{str(odom_tform_body)}" | ||
| ) | ||
|
|
||
| def set_initial_localization_fiducial(self, *args): |
bhung-bdai
reviewed
Aug 3, 2023
| # Downloading, reproducing, distributing or otherwise using the SDK Software | ||
| # is subject to the terms and conditions of the Boston Dynamics Software | ||
| # Development Kit License (20191101-BDSDK-SL). | ||
| def _get_localization_state(self, *args): |
There was a problem hiding this comment.
This function appears to be duplicated in wrapper.py
bhung-bdai
reviewed
Aug 3, 2023
| f.write(data) | ||
| f.close() | ||
|
|
||
| def _list_graph_waypoint_and_edge_ids(self, *args): |
There was a problem hiding this comment.
Almost exactly the same as the function in wrapper.py
bhung-bdai
reviewed
Aug 3, 2023
| "fiducial before attempting a navigation command." | ||
| ) | ||
|
|
||
| def _navigate_to(self, waypoint_id: str) -> typing.Tuple[bool, str]: |
There was a problem hiding this comment.
Again, similar duplication in wrapper.py
bhung-bdai
reviewed
Aug 3, 2023
| f.write(data) | ||
| f.close() | ||
|
|
||
| def _download_graph_and_snapshots( |
bhung-bdai
reviewed
Aug 3, 2023
| ) | ||
| return True, "Success" | ||
|
|
||
| def clear_graph(self) -> typing.Tuple[bool, str]: |
There was a problem hiding this comment.
Similar function without the extra nested function call in wrapper.py
Owner
Author
|
This was just a reference for me, please use rai-opensource#44 for comments. |
bhung-bdai
reviewed
Aug 3, 2023
| f"Got an error while clearing a graph and snanshots in a robot: {e}", | ||
| ) | ||
|
|
||
| def upload_graph(self, upload_path: str) -> typing.Tuple[bool, str]: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.