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

Remove reference to parentheses attribute from interchangeable complex type #12

Closed
mdgood opened this issue Aug 13, 2015 · 4 comments
Closed
Milestone

Comments

@mdgood
Copy link

mdgood commented Aug 13, 2015

Evan Brooks (@ebrooks01) found an error in the XSD documentation on the interchangeable complex type. The last sentence references a non-existent parentheses attribute and should be removed.

This is at line 2428 in the musicxml.xsd file in MusicXML 3.0. The DTD documentation does not have this error.

See the MusicXML forum discussion at http://forums.makemusic.com/viewtopic.php?f=12&t=2710.

@mdgood
Copy link
Author

mdgood commented Feb 8, 2016

The fix for this is in the description - the last sentence of the interchangeable complex type documentation should be removed. This is now on line 2431 of the current musicxml.xsd draft.

@lasconic
Copy link

lasconic commented Feb 8, 2016

You mean here

<xs:documentation>The interchangeable type is used to represent the second in a pair of interchangeable dual time signatures, such as the 6/8 in 3/4 (6/8). A separate symbol attribute value is available compared to the time element's symbol attribute, which applies to the first of the dual time signatures. The parentheses attribute value is yes by default.</xs:documentation>

See http://andrew.yurisich.com/work/2014/07/16/dont-link-that-line-number/

@mdgood
Copy link
Author

mdgood commented Feb 8, 2016

Thanks for the tip! I'll add links for the other documentation issues.

@mdgood mdgood closed this as completed Feb 10, 2016
@mdgood mdgood added this to the V3.1 milestone Jan 17, 2017
@mdgood
Copy link
Author

mdgood commented Dec 5, 2017

Pull request #117 addresses this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants