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

Fix ReadtheDoc build and breathe api generation #3

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

Briancbn
Copy link
Member

@Briancbn Briancbn commented Feb 8, 2024

This requires ros-industrial/easy_perception_deployment#68 to be merged first.

Right now it is linked to my fork with the Doxyfile.

This should fix the following

Signed-off-by: Chen Bainian <chenbn@artc.a-star.edu.sg>
@Briancbn Briancbn marked this pull request as draft February 8, 2024 04:10
Signed-off-by: Chen Bainian <chenbn@artc.a-star.edu.sg>
@carlowiesse
Copy link

carlowiesse commented Feb 15, 2024

The follwing 3 files don't have HPP files in the EPD repo anymore. Therefore, they won't be found by Doxyfile and the HTML will show an empty page with a warning.
Could you please remove them? @Briancbn

Signed-off-by: Chen Bainian <chenbn@artc.a-star.edu.sg>
@Briancbn
Copy link
Member Author

Removed as requested

I will change the epd source link once the code repo PR is merged.

@carlowiesse
Copy link

All good now, thanks @Briancbn

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