Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • Q quadruped-reactive-walking
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GepettoGepetto
  • quadruped-reactive-walking
  • Merge requests
  • !36

ROS integration for whole-body MPC

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Wilson Jallet requested to merge wjallet/ros-integration into wjallet/devel Mar 20, 2023
  • Overview 0
  • Commits 199
  • Pipelines 0
  • Changes 65

This PR adds ROS support for the whole-body MPC (my branch wjallet/devel). It eliminates a lot of dead code and makes necessary changes to some of the code architecture.

Changes:

  • simplify code which creates Crocoddyl OCP (ocp_croccodyl.py)
    • use numpy masks where applicable
    • rename ambiguous "task" variable
    • do not use np.insert or np.append
  • simplify the C++ Params class by:
    • removing a lot unused parameters (for centroidal MPC)
    • direct parsing of Yaml arrays to C++ Eigen vectors or matrices
    • remove or change type data members defined as std::vector only to be converted in Python to numpy
    • default constructor default-initializes all data members, does nothing else
    • static factory function create_from_file(filename=default_filename) replaces older ctor
    • add static factory create_from_str() which allows parsing from a std::string
  • enable pickling for the Params class (relevant for multiprocessing on macOS apparently)
  • fix the CMake config for macOS
  • fix package.xml
  • add CMake listfile formatting and linting
  • enable conversion from C++ YAML::Node to Python dicts (for Python-only params)
  • add utils for converting between ROS multiarrays and NumPy arrays
  • change the TaskSpec class:
    • some parameters are loaded from the Yaml config file
    • renames file problem_data.py to task_spec.py
  • rename CrocOCP to AlgtrOCP
    • separates AlgtrOCP into a subclass for instantiating ProxDDP, and one for proxddp.SolverFDDP
  • update the README
  • do not pass around the TaskSpec object
  • python: remove qrw.Utils.init_robot() because it modified the Params object without telling the user
    • replace by more explicit and self-contained make_footstep()
    • no attributes added to Params object (params.T is replaced by params.N_gait in C++ and initialized there)

We have tested building with Linux, macOS, and macOS with Catkin from robostack.

Thanks to the work of Etienne Arlaud @Inria Paris.

Resolves #3

Edited Mar 20, 2023 by Wilson Jallet
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: wjallet/ros-integration