Skip to content

Added torque-only client_command_mode.#304

Open
xmatousm wants to merge 3 commits intolbr-stack:rollingfrom
xmatousm:rolling
Open

Added torque-only client_command_mode.#304
xmatousm wants to merge 3 commits intolbr-stack:rollingfrom
xmatousm:rolling

Conversation

@xmatousm
Copy link

@xmatousm xmatousm commented Oct 9, 2025

Our proposal to torque-only command mode (related to #303). Tested on rolling and humble.

@mhubii
Copy link
Collaborator

mhubii commented Oct 29, 2025

sorry this went past my attention! Will review this next week.

@xmatousm
Copy link
Author

Never mind. Anyway, there was a bug about determining when the commands starts coming from the client. Hope now its fixed. Currently it is tested in humble, I will test this in rolling tomorrow.

@mhubii
Copy link
Collaborator

mhubii commented Oct 29, 2025

okay just had a brief look, looks pretty reasonable. Will try run this.

Thx for the effort

@mhubii
Copy link
Collaborator

mhubii commented Nov 6, 2025

Just FYI, trying to get #271 in first.

@mhubii
Copy link
Collaborator

mhubii commented Nov 24, 2025

Sorry for the delay here. Everything is now setup and I will look into getting this merged ASAP and don't want to block you here.

Just for my understanding and context, are you looking into using something like CRISP: https://github.com/utiasDSL/crisp_controllers?

@xmatousm
Copy link
Author

Maybe. I'm not familiar with the CRISP library, as in the project I'm working on (https://www.agimus-project.eu/) the solution for model predictive control is developped, which uses the effort-only interface of a robot.

@mhubii
Copy link
Collaborator

mhubii commented Nov 25, 2025

Okay got you. Just a quick question, did you run a load data calibration?

@xmatousm
Copy link
Author

Course I did. But in some situations the accuracy of the calibration is not enough - sometimes the stack of ros nodes and controllers starts too long that the robot unwanted motion during a "fall" is considerable (and the issue can be expected when robot with some payload in its gripper is e.g. emergency stopped)

@mhubii
Copy link
Collaborator

mhubii commented Nov 25, 2025

Course I did.

Got you, sorry just checking in case!

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.

2 participants