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

Version 1.1 -> 1.2 #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

stanbernsteen
Copy link
Collaborator

Updated CHANGELOG.md and README.md
Modified src/validator.py as follows:
To flag ordered lists (olo) as UnsupportedFeature instead of random bnode errors.
To flag properties not in the ontology as unknown property
Version 1.1 assumed an open model, where unknown properties are acceped as unconstrained.
Version 1.2 uses a less-open model, where
properties not in the ontology are rejected
properties not known to the class (but known to the ontology) are accepted as constrained.

Updated CHANGELOG.md and README.md
Modified src/validator.py as follows:
   To flag ordered lists (olo) as UnsupportedFeature instead of random bnode errors.
   To flag properties not in the ontology as unknown property
     Version 1.1 assumed an open model, where unknown properties are acceped as unconstrained.
     Version 1.2 uses a less-open model, where
          properties not in the ontology are rejected
          properties not known to the class (but known to the ontology) are accepted as constrained.
    In validator.py, replaced startswith(XSD) with startswith(str(XSD)).  This bug has already been fixed in the UCO github.
    In precondition.py, replaced len(alphabet) with len(alphabet)**prefix_length.  This is a new bug
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.

1 participant