Marco A. Gutierrez comments

Results 9 comments of


Marco A. Gutierrez

Adding thrust coefficient calculation

This got messed up, opened a new one [here](https://github.com/gazebosim/gz-sim/pull/1652) against Fortress as suggested.

[parsing] Script for converting model directives into sdformat files

It seems that the current script (https://github.com/RobotLocomotion/drake/pull/17223/commits/c3a34fb80b54581e19e1be55dc4b5dc137e537fe) generates the [valid output stated above](https://github.com/aaronchongth/drake/blob/aaron/convert-model-directives/multibody/parsing/test/convert_model_directives_test/valid_inject_frames.sdf). To confirm, I printed the `Bodies`, `Frames` and `Joints` for each `ModelInstance` for a `MultibodyPlant` loaded through...

Adding sdf::Errors as an error output option to PrintConfig methods

Due to code dependencies I will wait on https://github.com/gazebosim/sdformat/pull/1095 before updating this PR.

Adding sdf::Errors as an error output option to PrintConfig methods

Since https://github.com/gazebosim/sdformat/pull/1095 is in I went ahead and addressed the comments above.

Updating Rolling docs to Ubuntu Noble.

> Besides the change I put inline, we should also update the text in `Using-ros1_bridge-Noble-upstream.rst` to specify Ubuntu 24.04 Noble where appropriate. Changes added.

[parsing] Script for converting model directives into sdformat files

> I think there is another DMD feature not yet supported in this PR. See [#13065 (comment)](https://github.com/RobotLocomotion/drake/issues/13065#issuecomment-1273843047) for details or #17802 for the DMD feature. As per DM with @EricCousineau-TRI:...

[parsing] Script for converting model directives into sdformat files

@jwnimmer-tri I have no more cycles associated to TRI anymore, it would be great if you can finish whatever is needed to merge this.

SDF.cc update calls to use sdf::Errors output

This PR depends on https://github.com/gazebosim/sdformat/pull/1294 for the tests to pass since they check nothing is printed.

SDF.cc update calls to use sdf::Errors output

[This function signature](https://github.com/gazebosim/sdformat/blob/a01e8c4aefd07b7f1b6a01670779786c7887cbb9/src/XmlUtils.hh#L41) was kept (even though is not public) because otherwise it would involve further changes in `parser_urdf.cc` and we are leaving those files for future updates.