-
Notifications
You must be signed in to change notification settings - Fork 17
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
RSDK-6040 Add GetProperties endpoint to viam-cartographer #310
RSDK-6040 Add GetProperties endpoint to viam-cartographer #310
Conversation
go.mod
Outdated
@@ -329,3 +329,5 @@ require ( | |||
periph.io/x/conn/v3 v3.7.0 // indirect | |||
periph.io/x/host/v3 v3.8.2 // indirect | |||
) | |||
|
|||
replace go.viam.com/rdk => /Users/jeremyhyde/Development/rdk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[note] to be removed after viamrobotics/rdk#3376 and new rdk release
viam_cartographer.go
Outdated
@@ -678,3 +689,17 @@ func CheckQuaternionFromClientAlgo(pose spatialmath.Pose, componentReference str | |||
} | |||
return nil, "", errors.Errorf("error getting SLAM position: quaternion not given, %v", returnedExt) | |||
} | |||
|
|||
// checkCloseAndCloudStatus returns an error if the cartographer service has been previously closed or is being run in the cloud. | |||
func (cartoSvc *CartographerService) checkCloseAndCloudStatus(cmd string) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
func (cartoSvc *CartographerService) checkCloseAndCloudStatus(cmd string) error { | |
func (cartoSvc *CartographerService) isOpenAndRunningLocally(cmd string) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nit] think a name starting with is might read a little cleaner in all of the functions it is used in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 nits otherwise lgtm
Co-authored-by: kim-mishra <121991867+kim-mishra@users.noreply.github.com>
This PR implements Properties for viam-cartographer.
Related PR: viamrobotics/rdk#3376
JIRA Ticket: https://viam.atlassian.net/browse/RSDK-6040