-
Notifications
You must be signed in to change notification settings - Fork 29
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
Duplicated ClassCreation changes in CSV diff output #778
Comments
Note that it affects the YAML (and presumably the JSON) output as well: ---
id: uuid:ce46aef9-b1d4-47f1-8a59-4768fcb3974d
type: ClassCreation
about_node: CL:4023035
---
id: uuid:44669f60-4385-478e-aac7-e71254c8e80e
type: NewTextDefinition
new_value: A neuron that is derived from a precursor cell in the lateral ganglion
eminence.
about_node: CL:4023035
---
id: uuid:1c793c0b-b15c-4fe8-b682-db22bd206347
type: EdgeCreation
subject: CL:4023035
predicate: RO:0002202
object: UBERON:0004025
---
id: uuid:73860bfd-d3d2-4907-ae25-ed483316bb4d
type: EdgeCreation
subject: CL:4023035
predicate: rdfs:subClassOf
object: BFO:0000002
---
id: uuid:fc20817e-87ef-4718-b9cb-58dea0758535
type: EdgeCreation
subject: CL:4023035
predicate: rdfs:subClassOf
object: CL:0000540
---
id: uuid:8839f49f-a66f-4993-86f7-ceae127db17a
type: NewSynonym
new_value: LGE-derived neuron
about_node: CL:4023035
predicate: oio:hasExactSynonym
---
id: uuid:67b5ce02-2fa9-4358-b73d-cd610f870511
type: ClassCreation
about_node: CL:4023035
name: lateral ganglionic eminence derived neuron |
As for the KGCL output, there are no duplicated class creation changes, but there are no class creation changes at all. Instead, there are
|
@hrshdhgd let's fix on our 1-on-1 |
@gouttegd : v0.6.11 should fix this issue. Thanks for letting us know! |
Thanks for the quick fix! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Trying to generate a CSV-formatted diff with a command like:
yields a CSV file in which every
ClassCreation
-type changes are seemingly duplicated, as in this excerpt obtained from comparing http://purl.obolibrary.org/obo/cl/releases/2023-05-22/cl-base.obo and http://purl.obolibrary.org/obo/cl/releases/2024-04-15/cl-base.obo (here isolating all changes pertaining to CL:4023035):Notice the two
ClassCreation
changes (uuid:dad7e133-976c-40a5-98d0-a665312a7db4
anduuid:48e9e429-963c-4e67-8883-a3098b96e8e8
), which differs in that the latter is associated with aname
(“lateral ganglionic eminence derived neuron”).I believe either the second
ClassCreation
change should be aNodeRename
(“renaming” from nothing to the actual label) or the name should already be associated with the firstClassCreation
change and the second change should be pruned.Possibly related to #732 (but observed with OAK 0.6.10, which is supposed to have fixed that issue), though I have not looked at the underlying code.
The text was updated successfully, but these errors were encountered: