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

Allow ament-cmake-python build types #75

Merged
merged 3 commits into from
Mar 27, 2024

Conversation

rkent
Copy link
Contributor

@rkent rkent commented Mar 27, 2024

This enables displaying python API in a mixed module with C++.

You can see the effects in the full_package test case. That is, run the tests. Locate the test directory (which is typically at /tmp/pytest-of-), serve http from there (I use python -m http.server 8001), connect to localhost:8001 in your browser, and navigate to the latest full_package directory.

The index on the left should contain (and tests confirm) two items, "Python API" and "C++ API". Previously "C++ API" was expanded in the index to include the subitems "Class Hierarchy", "File Hierarchy", and "Reference". That only makes sense if the rosdoc2 output only contains the C++ API. It is intentional that those are now collapsed under a single heading "C++ API" We are adding other items there, now "Python API" and soon messages, standard documents, user documentation, and more, so we need to be clearer about what the items in the index are.

Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. This is great to add this scaling point. I manually tested it on a handful of packages.

@tfoote tfoote merged commit 7b364ef into ros-infrastructure:main Mar 27, 2024
2 checks passed
@kscottz
Copy link

kscottz commented Mar 27, 2024

@rkent these pull requests are great. We really appreciate it!

We should let the community know about the new features in rosdoc2. Would you be willing to make a post on ROS Discourse about these changes? I am more than happy to write the post, but I would prefer it come from the actual author. Please let me know.

@rkent
Copy link
Contributor Author

rkent commented Mar 27, 2024

@kscottz a post will be appropriate at some point, but right now this is a WIP so I don't really want anyone adapting to these changes yet. I did this work two years ago, but ultimately abandoned the effort because PR #28 which was a critical prerequisite just could not get landed. Reviews and landings are moving now again (thanks, @tfoote!), so I thought I would return to it.

You can see where this is going, at least what I previously implemented, at https://rosdabbler.github.io/fqdemo

We still need to land at least:

  • Standard documents
  • User documents
  • Meta-package dependencies

It might be appropriate do a post after all of these major pieces have landed. But there are still some serious questions about layout of how all this looks that could be addressed, and some larger issues I would like to address. So this is a moving target, but moving fairly quickly.

@kscottz
Copy link

kscottz commented Mar 28, 2024

@rkent I skimmed that repository and AFAICT it looks like you are trying to put together a high-quality template project? Is that a fair assessment? That's really exciting work and please keep us informed!

I was speaking to the improvements in ROSDoc2 in particular, not necessarily your downstream project.

I suggest that a post of the form, "I just added these new features to ROSDoc2, and I am working on this larger full quality project, would love some help / feedback" might be helpful. We're really trying to push on ROS documentation right now and this would really help us out. Happy to chat more; I would love to see your project through to completion.

@ros-discourse
Copy link

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/rosdoc2-undergoing-significant-change/36905/2

@rkent rkent deleted the allow-ament-cmake-python branch April 21, 2024 19:16
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.

4 participants