loco-3d / crocoddyl

Crocoddyl is an optimal control library for robot control under contact sequence. Its solver is based on various efficient Differential Dynamic Programming (DDP)-like algorithms
BSD 3-Clause "New" or "Revised" License
806 stars 166 forks source link

Muscodwalkin. Closes #72 - [merged] #444

Closed wxmerkt closed 4 years ago

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 7, 2019, 18:01

Merges muscodwalkin -> devel

Loads references from the centroidal solver, and runs ddp to track those references.

Some work still needs to be done to define properly the tasks (e.g., to add high cost velocity tracking on foot during contact transitions).

The current framework is meant to deal with multi-contact locomotion (and not just bipedal walking)

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 7, 2019, 18:05

added 22 commits

Compare with previous version

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 7, 2019, 18:07

changed title from Muscodwalkin to Muscodwalkin{+. Closes #72+}

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 7, 2019, 18:07

changed the description

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 7, 2019, 19:52

added 1 commit

Compare with previous version

wxmerkt commented 4 years ago

In GitLab by @nmansard on Feb 8, 2019, 08:56

I don't know what is the purpose of that, but I am sure I will loooove it when I will understand :)

wxmerkt commented 4 years ago

In GitLab by @nmansard on Feb 8, 2019, 08:57

namedtupple is what you want to use here.

wxmerkt commented 4 years ago

In GitLab by @nmansard on Feb 8, 2019, 08:59

Please remove the empty class. I am not against this kind of hacks if they save time, but don't put a hack in the main files (here in utils (utils.py does not equal hackcollection.py)).

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:18

This is purely cosmetic. Just so that there is a way to club objects which belong together, and not have to use dictionary. It isn't important to the PR.

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:21

changed this line in version 4 of the diff

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:21

added 5 commits

Compare with previous version

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:24

changed this line in version 5 of the diff

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:24

added 1 commit

Compare with previous version

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:24

I removed the EmptyClass.

wxmerkt commented 4 years ago

In GitLab by @proyan on Feb 8, 2019, 09:30

added 1 commit

Compare with previous version

wxmerkt commented 4 years ago

In GitLab by @nmansard on Feb 8, 2019, 10:31

resolved all discussions

wxmerkt commented 4 years ago

In GitLab by @nmansard on Feb 8, 2019, 10:31

merged

wxmerkt commented 4 years ago

In GitLab by @nmansard on Feb 8, 2019, 10:31

mentioned in commit 17e3fad6b68618f4c6d68695de50930cdde455f1