Mujoco 3.3.6 is breaking dm_control

[vmoens]

Installing dm_control from pypi makes it unusable:

$ suite.load("cheetah", "run")
...
  File "/Users/vmoens/venv/rl/lib/python3.11/site-packages/dm_control/suite/cheetah.py", line 45, in run
    physics = Physics.from_xml_string(*get_model_and_assets())
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/vmoens/venv/rl/lib/python3.11/site-packages/dm_control/mujoco/engine.py", line 444, in from_xml_string
    return cls.from_model(model)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/vmoens/venv/rl/lib/python3.11/site-packages/dm_control/mujoco/engine.py", line 427, in from_model
    return cls(data)
           ^^^^^^^^^
  File "/Users/vmoens/venv/rl/lib/python3.11/site-packages/dm_control/mujoco/engine.py", line 123, in __init__
    self._reload_from_data(data)
  File "/Users/vmoens/venv/rl/lib/python3.11/site-packages/dm_control/mujoco/engine.py", line 408, in _reload_from_data
    data=index.struct_indexer(self.data, 'mjdata', axis_indexers),)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/vmoens/venv/rl/lib/python3.11/site-packages/dm_control/mujoco/index.py", line 628, in struct_indexer
    attr = getattr(struct, field_name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'MjData' object has no attribute 'B_colind'

[yuvaltassa]

Thanks for catching. We'll fix

[yuvaltassa]

I believe this was fixed by the latest release. Please confirm.

For future reference, this is going to happen in the future. It is unfortunately impossible to release MuJoCo and dm_control at the same time, so breakages between releases of the former and latter are expected.

This is not to say that you shouldn't let us us know, you should, and thanks for that. Just that in a rather unfortunate sense this is WAI 🙂