-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update to handle constraints? #37
Comments
After discussing on the O&W call, we agreed to add a post-condition to fill that constraints be honored in any result. |
Will need to write up code snippets here that validates all constraints are satisfied (properties, categories, etc.). |
Perhaps just say: "Any constraints specified in the TRAPI must be respected by fill/lookup" in the description of the operation |
TODO: add "Any constraints specified in the TRAPI must be respected by fill/lookup" to the descriptions of fill, lookup, & lookup_and_score in a new PR |
Where should constraints go? Is it an option on our fill/bind operations? Subclasses thereof? Does everybody have to match the constraint? What if the tool lacks the data (like a node property) on which to perform the constraint?
The text was updated successfully, but these errors were encountered: