KodlabPenn / kodlab_mjbots_sdk

Repo for kodlab SDK for use with mjbots ecosystem
Other
8 stars 7 forks source link

MjbotsControlLoop should have init member function #23

Closed jdcaporale closed 2 years ago

jdcaporale commented 2 years ago

Currently any one off setup needs to be custom made into a constructor or put behind some sort of initial_call flag in CalcTorques(). MjbotsControlLoop should probably have a non-pure virtual function Init(), Begin(), or equivalent.

ethanmusser commented 2 years ago

There are some issues with putting virtual functions inside constructors. However, this virtual Init function could be called at the beginning of the Run function, which is where we currently do most of our intitialization.