-
Notifications
You must be signed in to change notification settings - Fork 84
#530, #529, #501, #494, #458 - Initial commit #532
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
Changes from all commits
4f3b3fd
2735608
bae38ea
2016a5c
8bb44b3
dcbd41e
d3b992e
b36bc1a
c29d3ed
b7f1766
86bc3e7
5dbff3d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Back of the envelope calculations (not including MRO):
The following should be removed:
http://www.ontologyrepository.com/CommonCoreOntologies/example_of_usage
http://www.ontologyrepository.com/CommonCoreOntologies/definition
FYSA In the merged and modal relations files:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@johnbeve Good catch. I think this file is regenerated anyway. I'll do that in next commit.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@neilotte Didn't realize you could see, I've been updating this comment so there might be a bit more to check (done with it now!)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@johnbeve I've updated the merged file, but I believe the dev team is automating generation of the modal file so I won't touch it further here (@mark-jensen or @cameronmore are welcome to though, here or on a subsequent MR).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mark-jensen @johnbeve Updated two sparql queries that used cco:definition as well.