Add part_of for discovered families of prefixes #1232
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds
part_of
relationships between prefixes and their assumed parents where missing. The PR stems from the discussion in #1222. I did the following:aop
foraop.events
,aop.stressor
, etc.) into families. Spot checking these relations show that these appear to be genuine families of prefixes, though there could be exceptions.part_of
relationship to the rootpart_of
relationship to a root and the root is also a Bioregistry prefix, add apart_of
relationship from the given prefix to the root prefix.Questions to consider:
part_of
relations to parents that aren't Bioregistry entries. An example that already exists ispubchem
.part_of
added here isn't actually correct, i.e., where two prefixes with.
-s in them share a root but shouldn't actually have apart_of
relationship to the same root?.
separation pattern which aren't picked up here - these might require manual curation.