-
Notifications
You must be signed in to change notification settings - Fork 6
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
Merge SKOS features and upstream #58
Conversation
c9c1a6c
to
b410981
Compare
6992eac
to
d51925f
Compare
module Concerns | ||
module Concept | ||
module Sort | ||
module ClassMethods |
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.
TDOD: do a pull request for this refactored module
@@ -889,7 +888,7 @@ def self.mappings_ont_build_query(class_id, page, size, sub1, sub2) | |||
#STRSTARTS is used to not count older graphs | |||
#no need since now we delete older graphs | |||
|
|||
filter += "\nFILTER (!STRSTARTS(str(?g),'#{ont_id}') || (?source = 'SAME_URI')" | |||
filter += "\nFILTER (!STRSTARTS(str(?g),'#{ont_id}')" |
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.
test if this works well
# re-generate metrics file | ||
submission.generate_metrics_file2(cls_metrics[:classes], indiv_count, prop_count, cls_metrics[:maxDepth]) | ||
submission.generate_metrics_file(cls_metrics[:classes], indiv_count, prop_count) |
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.
discuss with bioportal about this generate_metrics_file2 (method) for now stay with the old one
#cause it has not the subproperty added | ||
assert_equal 474, metrics.classesWithNoDefinition | ||
assert_equal 0, metrics.classesWithMoreThan25Children | ||
assert_equal 0, metrics.maxChildCount | ||
assert_equal 0, metrics.averageChildCount | ||
|
||
assert_equal 0, metrics.maxDepth |
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.
investigate this change
This reverts commit 79c9cd4.
d51925f
to
d43c850
Compare
Changes
SKOS scheme (f5b9f00)
SKOS collection (b410981)
SKOS roots, tree, and concept children filtered by collection and scheme
SKOS XL (79c9cd4)
Get the correct type for SKOS concepts (i.e skos:Concept, not owl:Class) (cfe3342)
Add tests for SKOS submissions, schemes, collections, skos_xl
Add SKOS links to the ontology model (3da053d)
Mappings (b567855)
Add dates attribute to Class/Concept module (29d8834)
Merge to upstream v5.23.1 (ba0b72b)
Extract concept sort module
Related branches