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

Improve anyStart schema #874

Merged
merged 19 commits into from
Nov 16, 2021
Merged

Conversation

bwbohl
Copy link
Member

@bwbohl bwbohl commented Nov 4, 2021

This PR improves the anyStart schema by:

  • adding @meiversion (actually att.meiVersion) to att.basic and thus allowing it on any element, and consequently to prevent duplicate attributes, removing it from the elements that could have @meiversionbefore.
  • adding a schematron rule to assert @meiversionis present on the root element

* fix `constraint/@scheme` to schematron instead of isoschematron
* redefine `@meiversion` to be recommended
* redefine `att.basic` to include `att.meiVersion`
* add constraintSpec to assert `@meiversion`on root-element
* remove specific reference to `att.meiVersion` from `meiCorpus`
* remove specific reference to `att.meiVersion` from `meiHead`
* remove specific reference to `att.meiVersion` from `music`
* fix `constraint/@scheme` to schematron instead of isoschematron
* redefine `@meiversion` to be recommended
* redefine `att.basic` to include `att.meiVersion`
* add constraintSpec to assert `@meiversion`on root-element
* remove specific reference to `att.meiVersion` from `meiCorpus`
* remove specific reference to `att.meiVersion` from `meiHead`
* remove specific reference to `att.meiVersion` from `music`
Copy link
Member

@rettinghaus rettinghaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@bwbohl bwbohl added this to 2021-11-26: ODD Friday in ODD Meetings Nov 6, 2021
customizations/mei-all_anyStart.xml Outdated Show resolved Hide resolved
customizations/mei-all_anyStart.xml Outdated Show resolved Hide resolved
* improve wording

Co-authored-by: Andrew Hankinson <andrew.hankinson@gmail.com>
@github-actions github-actions bot added the Component: Customizations changes to customizations/**/* (assigned automatically) label Nov 11, 2021
@ahankinson
Copy link
Member

Two approving reviews. Merging.

@ahankinson ahankinson closed this Nov 16, 2021
@ahankinson ahankinson reopened this Nov 16, 2021
@ahankinson ahankinson merged commit bcc0cc2 into music-encoding:develop Nov 16, 2021
@bwbohl
Copy link
Member Author

bwbohl commented Nov 16, 2021

Thx @ahankinson

@bwbohl bwbohl deleted the ftr_anyStart branch January 24, 2022 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Customizations changes to customizations/**/* (assigned automatically)
Projects
No open projects
ODD Meetings
  
2021-11-26: ODD Friday
Development

Successfully merging this pull request may close these issues.

None yet

3 participants