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

update tutorials #53

Merged
merged 6 commits into from
Jun 6, 2023
Merged

update tutorials #53

merged 6 commits into from
Jun 6, 2023

Conversation

andermi
Copy link
Collaborator

@andermi andermi commented Jun 3, 2023

update tutorials for new pack rate service and use non-blocking for python

andermi added 4 commits June 2, 2023 17:31
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
@github-actions
Copy link

github-actions bot commented Jun 3, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-06-06 19:25 UTC

@quarkytale
Copy link
Contributor

Changes look good to me, but the preview gives 404 for CppTemplate.md url seems to be old https://osrf.github.io/mbari_wec/pr-preview/pr-53/Tutorials/ROS2/CPPTemplate.md. Even when I manually updated its still gives 404. Refreshing didn't work. Same with CppOpenLoopControl.md and ClosedLoopControl.md.

@andermi
Copy link
Collaborator Author

andermi commented Jun 6, 2023

preview gives 404

Can you try again? It shows up fine for me:
https://osrf.github.io/mbari_wec/pr-preview/pr-53/Tutorials/ROS2/CppTemplate/

@andermi
Copy link
Collaborator Author

andermi commented Jun 6, 2023

all the links work for me as well

@quarkytale
Copy link
Contributor

Nope, tried it on another system too, the tutorial directs to CPPTemplate.md which gives the error and even if I manually change it to CppTemplate same error, something like this, enlarged content and 404 in the end:
buoy404_2
buoy404

@quarkytale
Copy link
Contributor

quarkytale commented Jun 6, 2023

Oh so when I try to go from the dropdown menu and click on Controller GitHub Template (C++) it creates a url with CPPTemplate.md but instead it should be CppTemplate/

Signed-off-by: Michael Anderson <anderson@mbari.org>
@quarkytale
Copy link
Contributor

I know what's wrong, you need to update mkdocs.yml as well to generate the correct navigation and links. Sorry I missed it in the review, all changes in tutorials.md should reflect in mkdocs.yml. I can make the quick change if you want.

@andermi
Copy link
Collaborator Author

andermi commented Jun 6, 2023

pushed a fix, thanks

@quarkytale
Copy link
Contributor

All good now, works for me!

@andermi
Copy link
Collaborator Author

andermi commented Jun 6, 2023

I also just fixed a reference to ignition gazebo and other broken links

@andermi andermi merged commit db48a1c into main Jun 6, 2023
@andermi andermi deleted the andermi/update_tutorials branch June 6, 2023 19:24
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