NASA Challenge_[@damianphim]_[Update Documentation] (closes #34)

[damianphim]

Closes #34

Demo and First-Time Building Overhaul

Previously, there was no documentation for any of the demos and the Space ROS documentation was outdated, so it took me like 3 days (embarrassing I know) to just build and get the demos running without any errors.

This PR updates the First-Time Building Space ROS page in the documentation (Tutorials -> First-Time Building Space ROS) to the latest method. In addition, it adds documentation for building and running MoveIt2, the Canadarm Demo, and the Curiosity Mars Rover demo.

I'm hoping this PR helps any future contributors not run into the same wall that I did!

[130s]

it took me like 3 days (embarrassing I know) to just build and get the demos running without any errors.

Not just you, I spent a day and haven't been able to see Mars rover on a simulator yet :/

I see this PR is updating .rst files. I asked why space_ros uses rst space-ros/space-ros#210 (I'm not against it, it just raises hurdle IMO).

[damianphim]

Not just you, I spent a day and haven't been able to see Mars rover on a simulator yet :/

Sounds like you made more progress than me on my first day, don't give up!

I see this PR is updating .rst files. I asked why space_ros uses rst space-ros/space-ros#210 (I'm not against it, it just raises hurdle IMO).

I believe the ROS 2 documentation also uses .rst files, which could be it.

[ivanperez-keera]

EDIT: I've applied this change to the PR.

@damianphim Can you please remove the file source/Getting-Started/.Building-From-Source.rst.swp?

This looks like a temporary (vim?) file that was introduced by mistake.

[ivanperez-keera]

I have not checked the accuracy of each step but, at first glance, this looks great and (almost) ready to merge, except for my two comments above.

Can I suggest @Bckempa to review the docker build steps, and @xfiderek to review the IKOS steps?

[ivanperez-keera]

EDIT: I've applied this change to the PR.

@damianphim Can you please also squash the two commits (since the second just fixes things about the first commit), and rephrase the commit message to be:

Create instructions for MoveIt2 image and two demos, and update first time build instructions (#34).

This commit creates documentation to build the MoveIt2 image, and to build and
run the Canadaarm and Mars Curiosity demos.

It also updates the First-Time Build Tutorial with build instructions, and
steps to run commands in a Space ROS image, to build the tests, and to run
generate the reports from IKOS.

Another suggestion, is the order of sections in the first time build tutorial ideal? Should connecting to a container come before generating tests? (it's a bit strange to me that it comes between the tests and IKOS).

[ivanperez-keera]

I've updated the PR with the changes I was proposing, leaving the original committer as author.

@Bckempa, @xfiderek, @asimonov, please let me know if anything needs to change. Otherwise we can merge this (with --no-ff) as is and close #34.

[Bckempa]

@ivanperez-keera, I've got journal and SBIR reviews occupying my next few evenings but I'll get to this before the weekend.

[asimonov]

I have looked through the changes and they look correct!