Merge branch 'sdf15' into arjo/fix/clone_parent_only

Author: scpeters

CMakeLists.txt

@@ -5,7 +5,7 @@ if(COMMAND CMAKE_POLICY)
   CMAKE_POLICY(SET CMP0004 NEW)
 endif(COMMAND CMAKE_POLICY)
 
-project (sdformat15 VERSION 15.0.0)
+project (sdformat15 VERSION 15.1.0)
 
 # The protocol version has nothing to do with the package version.
 # It represents the current version of SDFormat implemented by the software

Changelog.md

@@ -1,5 +1,34 @@
 ## libsdformat 15.X
 
+### libsdformat 15.1.0 (2024-11-13)
+
+1. Bazel CI
+    * [Pull request #1500](https://github.com/gazebosim/sdformat/pull/1500)
+
+1. Add bzlmod support to sdf15
+    * [Pull request #1493](https://github.com/gazebosim/sdformat/pull/1493)
+
+1. Support removing the actor, light, or model from the root
+    * [Pull request #1492](https://github.com/gazebosim/sdformat/pull/1492)
+
+1. Only look for psutil if testing is enabled
+    * [Pull request #1495](https://github.com/gazebosim/sdformat/pull/1495)
+
+1. Improve installation instructions
+    * [Pull request #1496](https://github.com/gazebosim/sdformat/pull/1496)
+
+1. Permit building python bindings separately from libsdformat library
+    * [Pull request #1491](https://github.com/gazebosim/sdformat/pull/1491)
+
+1. Change sdf\_config.h to sdf/config.hh everywhere
+    * [Pull request #1494](https://github.com/gazebosim/sdformat/pull/1494)
+
+1. Improve installation instructions
+    * [Pull request #1490](https://github.com/gazebosim/sdformat/pull/1490)
+
+1. Fix symbol checking test when compiled with debug symbols
+    * [Pull request #1474](https://github.com/gazebosim/sdformat/pull/1474)
+
 ### libsdformat 15.0.0 (2024-09-25)
 
 1. **Baseline:** this includes all changes from 14.5.0 and earlier.

include/sdf/Root.hh

@@ -246,6 +246,11 @@ namespace sdf
     public: sdf::ElementPtr ToElement(
         const OutputConfig &_config = OutputConfig::GlobalConfig()) const;
 
+    /// \brief Remove the actor, light, or model if one of them exists.
+    /// The SDF Root object can only hold one, or none, from the set
+    /// [Actor, Light, Model].
+    public: void ClearActorLightModel();
+
     /// \brief Private data pointer
     GZ_UTILS_IMPL_PTR(dataPtr)
   };

package.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <package format="2">
   <name>sdformat15</name>
-  <version>15.0.0</version>
+  <version>15.1.0</version>
   <description>SDFormat is an XML file format that describes environments, objects, and robots
 in a manner suitable for robotic applications</description>
 

python/src/sdf/pyRoot.cc

@@ -96,6 +96,10 @@ void defineRoot(pybind11::object module)
     .def("set_light", &sdf::Root::SetLight,
          "Set the light object. This will override any existing model, "
          "actor, and light object.")
+    .def("clear_actor_light_model", &sdf::Root::ClearActorLightModel,
+         "Remove the actor, light, or model if one of them exists."
+         "The SDF Root object can only hold one, or none, from the set"
+         "[Actor, Light, Model].")
     .def("add_world", [](Root &self, const World &_world)
          {
            ThrowIfErrors(self.AddWorld(_world));

python/test/pyRoot_TEST.py

@@ -358,5 +358,33 @@ def test_resolve_auto_inertials_with_save_calculation_configuration(self):
       self.assertEqual(len(inertialErr), 0)
       self.assertTrue(link.auto_inertia_saved())
 
+    def test_clear_actor_light_model(self):
+        root = Root()
+
+        # \TODO(anyone) Wrap the Actor class.
+        # self.assertEqual(None, root.actor())
+        # actor1 = Actor()
+        # actor1.set_name("actor1")
+        # root.set_actor(actor1)
+        # self.assertNotEqual(None, root.actor())
+        # root.clear_actor_light_model()
+        # self.assertEqual(None, root.actor())
+
+        self.assertEqual(None, root.light())
+        light1 = Light()
+        light1.set_name("light1")
+        root.set_light(light1)
+        self.assertNotEqual(None, root.light())
+        root.clear_actor_light_model()
+        self.assertEqual(None, root.light())
+
+        self.assertEqual(None, root.model())
+        model1 = Model()
+        model1.set_name("model1")
+        root.set_model(model1)
+        self.assertNotEqual(None, root.model())
+        root.clear_actor_light_model()
+        self.assertEqual(None, root.model())
+
 if __name__ == '__main__':
     unittest.main()

src/Root.cc

@@ -644,3 +644,9 @@ sdf::ElementPtr Root::ToElement(const OutputConfig &_config) const
 
   return elem;
 }
+
+/////////////////////////////////////////////////
+void Root::ClearActorLightModel()
+{
+  this->dataPtr->modelLightOrActor = std::monostate{};
+}

src/Root_TEST.cc

@@ -670,3 +670,45 @@ TEST(DOMRoot, WorldByName)
   ASSERT_TRUE(root.WorldNameExists("world2"));
   EXPECT_EQ("world2", root.WorldByName("world2")->Name());
 }
+
+/////////////////////////////////////////////////
+TEST(DOMRoot, ClearActorLightModel)
+{
+  sdf::Root root;
+  EXPECT_EQ(nullptr, root.Actor());
+  EXPECT_EQ(nullptr, root.Light());
+  EXPECT_EQ(nullptr, root.Model());
+
+  sdf::Actor actor1;
+  actor1.SetName("actor1");
+  root.SetActor(actor1);
+  EXPECT_NE(nullptr, root.Actor());
+  EXPECT_EQ(nullptr, root.Light());
+  EXPECT_EQ(nullptr, root.Model());
+  root.ClearActorLightModel();
+  EXPECT_EQ(nullptr, root.Actor());
+  EXPECT_EQ(nullptr, root.Light());
+  EXPECT_EQ(nullptr, root.Model());
+
+  sdf::Light light1;
+  light1.SetName("light1");
+  root.SetLight(light1);
+  EXPECT_EQ(nullptr, root.Actor());
+  EXPECT_NE(nullptr, root.Light());
+  EXPECT_EQ(nullptr, root.Model());
+  root.ClearActorLightModel();
+  EXPECT_EQ(nullptr, root.Actor());
+  EXPECT_EQ(nullptr, root.Light());
+  EXPECT_EQ(nullptr, root.Model());
+
+  sdf::Model model1;
+  model1.SetName("model1");
+  root.SetModel(model1);
+  EXPECT_EQ(nullptr, root.Actor());
+  EXPECT_EQ(nullptr, root.Light());
+  EXPECT_NE(nullptr, root.Model());
+  root.ClearActorLightModel();
+  EXPECT_EQ(nullptr, root.Actor());
+  EXPECT_EQ(nullptr, root.Light());
+  EXPECT_EQ(nullptr, root.Model());
+}