<div dir="ltr"><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div><div>Gerard/DMers,<br><br></div>With the couple element pattern restriction items which have come up recently and my experience working with the xslt script for Modelio, I decided to give the doc another full read/review.  Below are some comments.. many I believe have been mentioned before on an earlier draft.  Most are typographical or consistency items between text and figures.<br><br></div>VO-DML document: V1.0-20151007<br></div><div><br>pg 15:Figure 1<br></div>  + Contraint is not an extension of ReferableElement<br></div>  + missing SubsettedRole<br><br></div>pg 18:Section 4.1.1<br></div>  + pattern shown here (and reported by validator), does not match that <br></div>     shown in the VO-DML/Schema snippet for &quot;ElementID&quot; on pg 19<br></div>      &quot;[\w\._-]+&quot;  vs &quot;[\w\./_*]+&quot;<br><br></div>pg 18:Section 4.1.2<br></div>  + already reported separately, the pattern enforced by the validator <br></div>      does not match the definition of xsd:NCName, namely by the omission<br></div>      of &#39;.&#39; and &#39;-&#39; characters... possibly others<br><br></div>      It would be helpful to have a pattern or short description here.<br><br></div>pg 18:Figure 4<br></div>  + description ends mid-sentence.<br><br></div>pg 18: footnote.. &#39;will be left out in from now on&#39; =&gt; &#39;will be left out from now on&#39;<br><br></div>pg 20: VO-UML<br></div>  + &#39;This type has not explicit counterpart in VO-UML,&#39;  s/b<br></div><div>     &#39; This type has no explicit counterpart in UML,&#39;<br><br></div><div>pg 20: VO-DML/Schema<br></div><div>  + pattern shown here &#39;[\w_-]+:[\w\./_*]+&#39;<br></div><div>     a) indicates a pattern for the &#39;prefix&#39;, which is not stated in the Model.name description<br></div><div>         I&#39;m not sure what the validator enforces on this one.<br></div><div>     b) the ID pattern matche the previous schema snippet, not Section 4.1.1/validator pattern.<br><br><br></div><div>pg 22: VO-DML/XML<br></div><div>  + I&#39;ll mention it once here, but several of the XML examples don&#39;t include the closing tag, which they probably should, for completeness.. makes it clear that there is nothing missing.<br><br></div>pg 22: Section 4.4.1<br></div>  + does not mention the pattern restriction shown above in the Schema snippet<br></div>  + also, subsequent Schema snippet on pg 25 shows Model.name has type &quot;ModelPrefix&quot;<br></div>     which does not exist in the document.<br><br></div>pg 24: Figure 7<br></div>  + profile tags for model stereotype do not match text and what is being used in the IVOA_UML_Profile (&#39;title&#39;, &#39;version&#39;, &#39;authors&#39;, previousVersion&#39; )<br><br></div><div>pg 25: VO-DML/Schema<br></div><div>  + above mentioned.. Model element &#39;name&#39; has type &#39;ModelPrefix&#39; which is not in the document.<br><br></div><div>pg 26: Section 4.5.3<br></div><div>  + url type &#39;anyURL&#39; is &#39;anyURI&#39; in schema snippet of page 27<br><br></div><div>pg 27: Figure 8<br></div><div>  + modelimport stereotype tags do not match text and IVOA_UML_Profile.<br></div><div>     (version, url, documentationURL)<br></div><div>  + description states &#39;possible tags&#39;.. isn&#39;t this defining the tags?<br><br></div><div>pg 32: Section 4.9.1<br></div><div>pg 33: Section 4.10<br></div><div>  + could use some clarification of the pattern restriction in light of the list discussion.<br><br></div><div>pg 35: Figure 12<br></div><div>  + object has stereotype &lt;&lt;dataType&gt;&gt;, while description shows &lt;&lt;datatype&gt;&gt;<br><br></div><div>pg 39: VO-DML/Schema<br></div><div>  + &#39;... and multiplicity that defines whether.&#39;   ends mid-sentence.<br><br></div><div>pg 39: Section 4.14.1<br></div><div>  + element name == &#39;semanticconcept&#39;, but text references &#39;topconcept&#39;.<br></div><div>     If I&#39;m reading this right, the word &quot;topconcept&quot; is referring to this element name &quot;semanticconcept&quot;, and not the &#39;topConcept&#39; attribute on the SemanticConcept type.<br></div><div><br></div><div>pg 40: VO-DML/Schema<br></div><div>  + related to above, the element name should be &#39;semanticconcept&#39;<br><br></div><div>pg 42: top of page<br></div><div>  + large space in final sentence.<br><br></div><div>pg 61: Appendix C<br></div><div>  + &#39;for generating such unique [word missing], and the following grammar is built starting from [such s/b &#39;this&#39;?] previous attemp&#39;<br><br><br></div></div>