Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sphinx extension for autodocs #97

Merged

Conversation

christophfroehlich
Copy link
Contributor

@christophfroehlich christophfroehlich commented May 18, 2023

I created a sphinx extension, which uses the python scripts of the generate_parameter_library to create parameter documentation automatically.

It can be configured in any rst file by adding the directive

.. generate_parameter_library_details:: ../src/admittance_controller_parameters.yaml

for a detailed parameter list and

.. generate_parameter_library_default:: ../src/admittance_controller_parameters.yaml

for a default ros-params yaml file.

An example with the current rst template
image

Open tasks:

@christophfroehlich christophfroehlich marked this pull request as ready for review May 20, 2023 07:58
@destogl
Copy link
Member

destogl commented Jun 4, 2023

@christophfroehlich should we merge this or wait that we can also generate __map__ parameters?

@christophfroehlich
Copy link
Contributor Author

I think we can merge it, because ideally we don't have to change this extension but only wait for a new release of the generate_parameter_library. And we already can use it for yaml-files without __map_s

@christophfroehlich christophfroehlich merged commit b8bf31a into ros-controls:master Jun 4, 2023
@christophfroehlich christophfroehlich deleted the ext_parameter branch June 4, 2023 14:48
@christophfroehlich
Copy link
Contributor Author

Damn, the action fails on the master branch. I'll have a look how to fix that

@christophfroehlich
Copy link
Contributor Author

@Mergifyio backport humble

@mergify
Copy link
Contributor

mergify bot commented Jun 12, 2023

backport humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jun 12, 2023
christophfroehlich added a commit that referenced this pull request Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants