Skip to content

Conversation

@davidetome
Copy link
Contributor

This is a draft PR to discuss changes in order to document the migration to motor control v6.

@valegagge the body_part--ebX-jA_B-mc.xml was already aligned to body_part--ebX-jA_B-mc.xml.txt a part of some comments added in the first commit.

I just added the groups for each board described in Tutorial.txt

cc @pattacini @ale-git

@davidetome davidetome changed the base branch from master to devel March 15, 2022 13:39
@pattacini
Copy link
Member

Hi @valegagge

Could you please review the PR?

@davidetome
Copy link
Contributor Author

just spoke w/ @valegagge , fixed the template as decided, we'll go on w/ the wiki mods

@valegagge
Copy link
Member

valegagge commented Mar 16, 2022

Perfect!
As discussed with @davidetome, we still have some doubts about the velocity PID configuration. I'd like to deepen with @ale-git before merging this PR.

@davidetome
Copy link
Contributor Author

Ciao @valegagge. please take a look at the diagrams
Presentation3.pptx

@valegagge
Copy link
Member

Ciao @valegagge. please take a look at the diagrams Presentation3.pptx

Great! We checked the diagrams together.

@valegagge
Copy link
Member

After a chat with @ale-git, I updated the motor control config file accordinglyhere.
@davidetome if you are agree, you can merge my PR and then merge this PR.

Updated  body_part--ebX-jA_B-mc.xml of mc version 6
@pattacini pattacini marked this pull request as ready for review March 19, 2022 09:05
Copy link
Member

@pattacini pattacini left a comment

Choose a reason for hiding this comment

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

Thanks heaps @davidetome and @valegagge for this contribution!

@pattacini pattacini merged commit 99fd14e into robotology:devel Mar 19, 2022
@pattacini pattacini linked an issue Mar 19, 2022 that may be closed by this pull request
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.

Update wiki and templates

3 participants