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

Migration guide from ROS Wiki to docs.ros.org #4167

Merged
merged 11 commits into from Feb 16, 2024
Merged

Conversation

kscottz
Copy link
Collaborator

@kscottz kscottz commented Feb 14, 2024

This pull request adds a short guide on how to migrate a single page from the ROS Wiki to docs.ros.org.

I am working on a plan to sunset the ROS Wiki. I wanted to get a better sense of how best to migrate a pages over. Long story short, I think the best approach for most pages is to manually download a given page and convert it with PanDoc. While this approach has a lot of warts, I think it is probably the best approach as it forces you to manually update / validate every link in the page.

I ran through this approach on a few trivial pages to test it for basic functionality. Once this pull request is merged I intend to migrate a few key pages over.

I realize this guide is quite long. If you would like me to break it out into its own page just let me know.

Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

I've left a number of changes inline.

I'm torn about whether to put this in another page. This document is already pretty long, so I don't love adding to it. But it also doesn't really feel natural to make a new page. For now, I'm OK with just adding this to the existing page.

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

a few cosmetic suggestions.

@ros-discourse
Copy link

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

https://discourse.ros.org/t/growing-issue-with-ros-documentation/36075/12

kscottz and others added 7 commits February 14, 2024 16:15
…umentation.rst

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
…umentation.rst

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
…umentation.rst

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
…umentation.rst

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
…umentation.rst

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
…umentation.rst

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
…umentation.rst

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Copy link
Contributor

@mikeferguson mikeferguson left a comment

Choose a reason for hiding this comment

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

Saw this linked on discourse - looks good. A few other notes:

  • If you're looking for an example of a package/repo that has been ported from ROS Wiki to rosdoc2/api docs - I present http://docs.ros.org/en/rolling/p/image_pipeline/ - I just finished porting about 98% of the wiki content last week.
  • One of the painful parts I ran into was that the rate limiting on the wiki meant I could only download like 2-3 images at a time.... and there were about 30 in the wiki pages

#. Find and download any images that may be in the old document. The easiest way to do this is to right click in the browser and download all of the images. Alternatively you can find images by searching for ``<img src>`` tags in the HTML file.

#. For each image files downloaded update the image file links to point to the correct image directory for the ROS Docs.
If any of the images require updating, or could be replaced with a `Mermaid <https://mermaid.js.org/intro/>`__ chart, please make this change.
Copy link
Contributor

Choose a reason for hiding this comment

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

does mermaid work for all packages (i.e. generically with rosdoc2 for api docs?) or just ros core documentation? If just core docs, might want to clarify that

Copy link
Contributor

Choose a reason for hiding this comment

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

does mermaid work for all packages (i.e. generically with rosdoc2 for api docs?) or just ros core documentation? If just core docs, might want to clarify that

Good point. Right now, it will only work for the ROS 2 core documentation, so we should be explicit here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@mikeferguson you're a saint. We really appreciate it. I guess we could include a longer example.

I don't have a great answer on wiki images. The wiki is held together with tape and bailing wire at this poing. I was considering making a markdown clone of the wiki but I fear it might take the whole edifice to the ground. More to the point, the path the wiki uses for images is a bit weird, I ran into images trying use wget to pull images out the html files.

Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

Other than @mikeferguson 's suggestions, this looks good to me.

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

This looks good to me. Thanks @kscottz for putting this together, and thanks @mikeferguson for beta-testing this, sort of.

@clalancette clalancette added the backport-all backport at reviewers discretion; from rolling to all versions label Feb 16, 2024
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette clalancette merged commit 8fbb74d into rolling Feb 16, 2024
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the migration-guide branch February 16, 2024 14:07
mergify bot pushed a commit that referenced this pull request Feb 16, 2024
* Migration guide.

Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 8fbb74d)

# Conflicts:
#	source/The-ROS2-Project/Contributing/Contributing-To-ROS-2-Documentation.rst
mergify bot pushed a commit that referenced this pull request Feb 16, 2024
* Migration guide.

Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 8fbb74d)

# Conflicts:
#	source/The-ROS2-Project/Contributing/Contributing-To-ROS-2-Documentation.rst
clalancette pushed a commit that referenced this pull request Feb 16, 2024
* Migration guide.

Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 8fbb74d)
clalancette pushed a commit that referenced this pull request Feb 16, 2024
* Migration guide.

Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 8fbb74d)
clalancette pushed a commit that referenced this pull request Feb 16, 2024
* Migration guide.

Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 8fbb74d)

Co-authored-by: Katherine Scott <katherineAScott@gmail.com>
clalancette pushed a commit that referenced this pull request Feb 16, 2024
* Migration guide.

Signed-off-by: Katherine Scott <katherineAScott@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 8fbb74d)

Co-authored-by: Katherine Scott <katherineAScott@gmail.com>
@ros-discourse
Copy link

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

https://discourse.ros.org/t/ros-news-for-the-week-of-february-12th-2024/36135/1

@ros-discourse
Copy link

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

https://discourse.ros.org/t/discuss-tools-and-best-practices-for-3d-robot-assets-urdf-sdformat-cad-etc/36997/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-all backport at reviewers discretion; from rolling to all versions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants