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

Test both pr1012 and pr1040 #18

Open
wants to merge 9 commits into
base: wide_stereo_camera_info
Choose a base branch
from

Conversation

YutoUchimi
Copy link

Related to jsk-ros-pkg#426

k-okada and others added 9 commits December 15, 2019 09:24
… between robot and versions

use python command to convert from robot_description string format to xml format, use lxml because of pretty function https://www.ronrothman.com/public/leftbraned/xml-dom-minidom-toprettyxml-and-silly-whitespace/, instead of minidom or ElementTree

```
python -c "import yaml; import datetime; from lxml import etree; comment=etree.Comment(datetime.datetime.today().strftime(' generated from pr1040 on %Y/%m/%d ')); urdf=etree.XML(yaml.load(open('pr1040_urdf.yaml').read())); print(etree.tostring(comment, pretty_print=True, xml_declaration=True)); print(etree.tostring(urdf, pretty_print=True, xml_declaration=False));"
```
@k-okada
Copy link
Owner

k-okada commented Dec 21, 2019

thank you , I just want to know jsk-ros-pkg#426 respect camera_info for each robot, but do not care about the joint offset. Is it ok for you?

@YutoUchimi
Copy link
Author

Yes, all I want to do is to fix camera_info.

@k-okada k-okada force-pushed the wide_stereo_camera_info branch from 17fbe8c to c1b65f4 Compare November 12, 2021 06:19
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