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

Define the production rules for SVRL #7

Open
xatapult opened this issue Jun 10, 2021 · 4 comments
Open

Define the production rules for SVRL #7

xatapult opened this issue Jun 10, 2021 · 4 comments
Labels
2025 A change made in preparing the 2025 edition clarification Discusses and clarifies ambiguities SVRL Issues relating to SVRL

Comments

@xatapult
Copy link
Collaborator

xatapult commented Jun 10, 2021

There is a schema for SVRL but there are no "production rules". What I mean is that it is not defined that when a pattern gets active a <svrl:active-pattern .../> is produced. And when a rule fires <svrl:fired-rule .../>. Etc.

Of course it's obvious, but a specification should IMHO not leave things to being obvious. It should define the exact rules for producing an SVRL document based on a Schematron validation run.

@dmj dmj added the clarification Discusses and clarifies ambiguities label Aug 30, 2021
@AndrewSales
Copy link
Collaborator

SVRL appears in an informative part of the standard, but agree that more clarity would help.

@rjelliffe
Copy link
Member

rjelliffe commented Apr 26, 2023 via email

@AndrewSales AndrewSales added the SVRL Issues relating to SVRL label Jun 19, 2023
AndrewSales added a commit to Schematron/schema that referenced this issue Mar 9, 2024
add optional role attribute to pattern and group, to facilitate svrl:active-*/@ROLE;
add optional role attribute to svrl:diagnostic-reference, in line with svrl:property-reference
re Schematron/schematron-enhancement-proposals#5, Schematron/schematron-enhancement-proposals#7
@AndrewSales
Copy link
Collaborator

AndrewSales commented Mar 9, 2024

@rjelliffe - a few notes about the mapping.

For diagnostic, "@role else @id" is given. @id is a required attribute, so I propose adding diagnostic/@role to bring it into line with property in this respect and so map it to svrl:diagnostic-reference/@role instead.

For p, svrl:text has no @class or @id. Also, my reading is it can map to svrl:text only within schema, since active-pattern and fired-rule are empty and phases don't have an SVRL analogue.

For span, emph and dir, only span can have @class in a schema at present.

@AndrewSales
Copy link
Collaborator

Annex D now includes a mapping, also highlighting where none exists.

@AndrewSales AndrewSales added the 2025 A change made in preparing the 2025 edition label Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2025 A change made in preparing the 2025 edition clarification Discusses and clarifies ambiguities SVRL Issues relating to SVRL
Projects
None yet
Development

No branches or pull requests

4 participants