From cefc290eac996ed9c6a0b1d19216cae6ae7d3d70 Mon Sep 17 00:00:00 2001
From: Saran Tunyasuvunakool <stunya@google.com>
Date: Thu, 13 Apr 2023 05:21:26 -0700
Subject: [PATCH] Reorder arguments of `Simulate::Load` in Python to match C++.

PiperOrigin-RevId: 523973129
Change-Id: I58760e3e392a208dd8257c340da146aa8f745fb7
---
 python/mujoco/simulate.cc | 6 +++---
 python/mujoco/viewer.py   | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/python/mujoco/simulate.cc b/python/mujoco/simulate.cc
index 728b052b..1b43f167 100644
--- a/python/mujoco/simulate.cc
+++ b/python/mujoco/simulate.cc
@@ -51,13 +51,13 @@ PYBIND11_MODULE(_simulate, pymodule) {
             std::make_unique<mujoco::GlfwAdapter>());
       }))
       .def(
-          "renderloop",
+          "render_loop",
           [](mujoco::Simulate& simulate) { simulate.RenderLoop(); },
           py::call_guard<py::gil_scoped_release>())
       .def(
           "load",
-          [](mujoco::Simulate& simulate, const std::string& path,
-             MjModelWrapper& m, MjDataWrapper& d) {
+          [](mujoco::Simulate& simulate, MjModelWrapper& m, MjDataWrapper& d,
+             const std::string& path) {
             simulate.Load(m.get(), d.get(), path.c_str());
           },
           py::call_guard<py::gil_scoped_release>())
diff --git a/python/mujoco/viewer.py b/python/mujoco/viewer.py
index b5d30642..e6ce21e4 100644
--- a/python/mujoco/viewer.py
+++ b/python/mujoco/viewer.py
@@ -273,7 +273,7 @@ def _launch_internal(model: Optional[mujoco.MjModel] = None,
   atexit.register(exit_requester)
 
   side_thread.start()
-  simulate.renderloop()
+  simulate.render_loop()
   atexit.unregister(exit_requester)
   side_thread.join()