Open cardboardcode opened 4 years ago
Thanks for reporting an issue. We will have a look asap. If you can think of a fix, please consider providing it as a pull request.
Validated
The section below lists what are the potential edits required to correct and/or improve the documentation for the MoveIt Melodic Tutorials.
Each edit is corresponded to the specific Section or Section header.
Add full stop (.) at the end of the statement description smoothness_cost_weight under Section header, Tweaking some of the parameters for CHOMP. [Rationale] - Sentence structure error.
Place in bullet point for the unused/commented parameters under Section header, Tweaking some of the parameters for CHOMP. [Rationale] - Clearer format for easy reading.
Bold the parameter variables under Section header, Tweaking some of the parameters for CHOMP. [Rationale] - Clearer format for easy reading.
Put in bullet point for the Strengths of STOMP under Section Header, STOMP Planner. [Rationale] - Clearer format for easy reading.
Fix more info
hyperlink under Section Header, STOMP Planner.
[Rationale] - For providing proper link to compliment tutorial information.
Bold the parameter variables under Section header, Tweaking some of the parameters for STOMP. [Rationale] - Clearer format for easy reading.
Remove STOMP
after first line in content under Section Header, Difference between plans obtained by STOMP, CHOMP and OMPL
[Rationale] - Trivial typo error.
Clarification of phrase within reasonable times under Section Header, STOMP Planner [Rationale] - Would provide a more concrete image of what level of proficiency STOMP operates in.
The rest of the sections validated do not require further edits.
MoveIt Tutorials Sections Validated
Perception Pipeline Tutorial
-> Detecting and Adding Object
Replace rviz with RViz [Rationale] - Be consistent with naming conventions.
Transform cache issue - is this updated and resolved by @Ridhwanluthra & @MohmadAyman ?
Have gone through Perception pipeline until OMPL Planner, seems to be fine.
Thanks @cardboardcode and @sheikspeare for validating the tutorials.
Could you, please, directly file PRs with the proposed changes? The tutorials reside in doc/*/*.rst
files in the repo https://github.com/ros-planning/moveit_tutorials. You can even edit those files directly on github (and preview your changes). Thanks a lot!
@rhaschke
Hi Mr Haschke, noted. :smile:
While I wait for the rest of my team to validate the rest of the MoveIt tutorials, I will compile the edit suggestions here and make an overall Pull Request (PR).
Will update again once all the validation has been done.
Thanks for validating these!
Let's also make sure whatever changes get merged into melodic, also get merged into the master
branch where active development is.
MoveIt Tutorials Sections Validated
Getting Started
MoveIt Quickstart in RViz
Move Group C++ Interface
-> Moveit Quickstart in RViz and Move Group C++ Interface
MoveIt Tutorial Sections Validated:
The documentation is fine. No fault found in this sections.
MoveIt Tutorial Sections Validated
The documentation is alright. No errors found in these sections.
Why?
Similar to Issue #645, this issue thread is created to facilitate checks for existing Getting Started or Advanced MoveIt Tutorials specifically for the version ROS Morenia Melodic
While the issue linked is similar, it seems it is specific to the ROS Kinetic Kame version of the MoveIt Tutorials. As @davetcoleman mentioned in this comment, more testing is needed for Melodic.
Purpose?
Highlight as much spelling errors, code syntax errors spotted in the documentation as possible. The assignee (probably me) will collate all the needed changes and make a pull request.
Link
https://docs.ros.org/melodic/api/moveit_tutorials/html/index.html