Skip to content

Conversation

SprGrf
Copy link
Contributor

@SprGrf SprGrf commented Jul 30, 2025

Description

Includes the dashboard controller example. Many TODOs and comments left inside but, as discussed, it's better to start reviewing before it gets too large.

Review guidelines

Estimated Time of Review: 15 minutes

Checklist before merging:

  • Confirm that the relevant changelog(s) are up-to-date in case of any user-facing changes

@SprGrf SprGrf requested review from bpapaspyros, domire8 and yrh012 July 30, 2025 16:16
@SprGrf SprGrf requested a review from eeberhard as a code owner July 30, 2025 16:16
Copy link
Member

@bpapaspyros bpapaspyros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some first comments, structure looks good to me though, we can keep adding content


# Hardware Interface

TBD: There is already a [page](../../concepts/building-blocks/hardware-interfaces.md) on hardware interfaces. Are these
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok with minor duplication here and there, I think it's quite likely people land in this page so giving a direction to the original page would make sense. That is, perhaps we link to that page, but it's ok to have the current text too

@SprGrf SprGrf changed the title docs: UR hardware interface guide - Dashboard controller docs: UR hardware interface guide - Dashboard & Hand Guiding controller Jul 31, 2025

## Hand Guiding controller

NOTE: Did not really know what to write in the next paragraph, the reason we did not just implement simple freedrive.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason we implemented the hand guiding controller is that the freedrive feels a bit janky sometimes, whereas the admittance control is much smoother and yields nicer trajectories/recordings

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't want to phrase it like that, seems like we are trash-talking UR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I rephrased a bit.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean fair enough but we also had a lot of conversations with people from UR where they admitted that this is not usable and they'd like to make it better

@domire8
Copy link
Member

domire8 commented Aug 8, 2025

I was inspired in the airplane and had a go at this. Rewrote the first part of the guide until

Run an AICA application as one node of a program

Copy link
Member

@bpapaspyros bpapaspyros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for keeping this up. I suppose there are still some todos to be resolved, but seems on track

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.

3 participants