Added the option to set node name parameter from CLI

Signed-off-by: Ricardo Manríquez <[email protected]>

Author: Ricardo Manríquez

ros2bag/ros2bag/verb/record.py

@@ -174,6 +174,10 @@ def add_arguments(self, parser, cli_name):  # noqa: D102
             '--use-sim-time', action='store_true', default=False,
             help='Use simulation time.'
         )
+        parser.add_argument(
+            '--node-name', type=str, default='rosbag2_recorder',
+            help='Specify the recorder node name. Default is rosbag2_recorder.'
+        )
         self._subparser = parser
 
     def main(self, *, args):  # noqa: D102
@@ -259,7 +263,7 @@ def main(self, *, args):  # noqa: D102
         recorder = Recorder()
 
         try:
-            recorder.record(storage_options, record_options)
+            recorder.record(storage_options, record_options, args.node_name)
         except KeyboardInterrupt:
             pass
 

rosbag2_py/src/rosbag2_py/_transport.cpp

@@ -358,9 +358,7 @@ PYBIND11_MODULE(_transport, m) {
   py::class_<rosbag2_py::Recorder>(m, "Recorder")
   .def(py::init())
   .def(
-    "record", &rosbag2_py::Recorder::record,
-    py::arg("storage_options"),
-    py::arg("record_options"),
+    "record", &rosbag2_py::Recorder::record, py::arg(), py::arg(),
     py::arg("node_name") = "rosbag2_recorder")
   .def("cancel", &rosbag2_py::Recorder::cancel)
   ;