Skip to content

Change link description to use choice instead of all.#97

Open
dljsjr wants to merge 1 commit into
ros:masterfrom
dljsjr:master
Open

Change link description to use choice instead of all.#97
dljsjr wants to merge 1 commit into
ros:masterfrom
dljsjr:master

Conversation

@dljsjr

@dljsjr dljsjr commented Jan 24, 2017

Copy link
Copy Markdown

See #95

Unbounded elements in all are fine in XSD 1.1 but not valid in XSD 1.0.
There aren't a ton of technologies out in the wild that leverage XSD 1.1
(e.g. code generators). This definition should be acceptable for
validation of XML documents but I'm not convinced it's a nice way to do
things re: code generation, as the code generated by XJC for example
contains a single List field containing the inertial and the
collisions/visuals.

Unbounded elements in all are fine in XSD 1.1 but not valid in XSD 1.0.
There aren't a ton of technologies out in the wild that leverage XSD 1.1
(e.g. code generators). This definition should be acceptable for
validation of XML documents but I'm not convinced it's a nice way to do
things re: code generation, as the code generated by XJC for example
contains a single List field containing the inertial and the
collisions/visuals.
@scpeters

Copy link
Copy Markdown
Contributor

@clalancette @sloretz do you know if ROS uses this schema file? I'd like to test and merge this

@sloretz

sloretz commented Oct 26, 2017

Copy link
Copy Markdown
Contributor

@scpeters I looked around and didn't find anything using this schema file. Looking around this repo, it appears urdf.xsd isn't installed. Someone using it would have had to copy it from source. Edit I'm not sure how this is packaged but urdfdom-0.4.1 has the schema in its archive http://archive.ubuntu.com/ubuntu/pool/universe/u/urdfdom/urdfdom_0.4.1.orig.tar.gz

Interestingly there are a couple examples of people writing their own schema.

One uses <all> but limits <visual> and <collision> to 1 occurrence. The other uses a sequence and forces children of <link> to be ordered.

@aminya

aminya commented May 5, 2022

Copy link
Copy Markdown
Contributor

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.

4 participants