Skip to content

Conversation

@130s
Copy link
Contributor

@130s 130s commented Jun 13, 2017

I believe how to use moveit/moveit#491 isn't documented anywhere yet (took me awhile to figure it out).

Should be portable to indigo-devel. In fact I took the snapshot on Indigo.

@v4hn
Copy link
Contributor

v4hn commented Jun 13, 2017

@130s testing failed

@130s
Copy link
Contributor Author

130s commented Jun 13, 2017

https://travis-ci.org/ros-planning/moveit_tutorials/builds/242549355#L976

The command "sphinx-build -W -b html . native_build" exited with 1.
0.19s$ rosdoc_lite -o build .
Traceback (most recent call last):
  File "/opt/ros/indigo/bin/rosdoc_lite", line 37, in <module>
    rosdoc_lite.main()
  File "/opt/ros/indigo/lib/python2.7/dist-packages/rosdoc_lite/__init__.py", line 225, in main
    pkg_desc = get_pkg_desc(path)
  File "/opt/ros/indigo/lib/python2.7/dist-packages/rosdoc_lite/__init__.py", line 241, in get_pkg_desc
    pkg_desc = packages.parse_package(path)
AttributeError: 'module' object has no attribute 'parse_package'

I think this is related to changes somewhere upstream and in rosdoc_lite it's already addressed (e.g. ros-infrastructure/rosdoc_lite#74) but its new version that includes the fixes are not yet synced for Indigo as of today. Indigo sync happened 2 weeks ago so hopefully next sync will be made soon. Then we can restart the CI job.

@v4hn
Copy link
Contributor

v4hn commented Jun 14, 2017

Oh, it's two errors then. I didn't see the one you pointed out :D

I saw only the other one at first glance:

/home/travis/build/ros-planning/moveit_tutorials/doc/ros_visualization/visualization_tutorial.rst:168: WARNING: Inline interpreted text or phrase reference start-string without end-string.

@130s 130s force-pushed the k/rviz_add_moveit_491 branch from 23e040c to e9c50aa Compare June 14, 2017 19:09
@130s
Copy link
Contributor Author

130s commented Jun 14, 2017

@v4hn Thanks for catching that. Now it should be fixed.

For rosdoc_lite, I asked for when's the next sync.

@v4hn v4hn merged commit 82e5142 into moveit:kinetic-devel Jun 19, 2017
@v4hn
Copy link
Contributor

v4hn commented Jun 19, 2017

I don't want to have this hanging here until upstream fixed the package, so I'll merge it.

@130s 130s deleted the k/rviz_add_moveit_491 branch June 19, 2017 15:18
@davetcoleman
Copy link
Member

Great documentation, thanks!

130s added a commit that referenced this pull request Jun 22, 2017
@130s
Copy link
Contributor Author

130s commented Jun 22, 2017

Ported to indigo-devel 5a3816b

davetcoleman pushed a commit to davetcoleman/moveit_tutorials that referenced this pull request Oct 19, 2017
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.

3 participants