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

Add autowalk client methods #95

Open
wants to merge 3 commits into
base: spot_arm
Choose a base branch
from

Conversation

sktometometo
Copy link

Add eus interface to k-okada/spot_ros-arm#6

Please use this branch if you want to use AutoWalk.

@sktometometo sktometometo changed the base branch from master to spot_arm April 17, 2023 03:56
@sktometometo
Copy link
Author

@k-okada Could you merge this for my experiments?

@sktometometo sktometometo force-pushed the PR/spot_arm/add-autowalk-client-methods branch from a48163a to 497571b Compare May 25, 2023 02:03
@tkmtnt7000
Copy link
Collaborator

I think it is OK to merge for your experiment.
This PR will widen usecases for navigation.

@k-okada
Copy link
Owner

k-okada commented May 25, 2023

the target of this PR should be sktometometo:PR/arm/update-autowalk-interfaces ?

sorry, this is not spot_ros repository.

  • This PR will widen usecases for navigation.

Please list of your usecases. @tkmtnt7000

@tkmtnt7000
Copy link
Collaborator

Sorry, I misunderstood the purpose of this PR. I thought this pr added graph navigation system that was used in my bthesis trash bag carrying demo... So,

This PR will widen usecases for navigation.

is not appropriate...

But if autowalk graph can be loaded faster, it may be a nice feature. I have encountered the long waiting for demo starting or graph changing when I loaded large autowalk graph.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants