Skip to content
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

Accidental equivalent definitions #1069

Closed
matentzn opened this issue Mar 11, 2021 · 16 comments
Closed

Accidental equivalent definitions #1069

matentzn opened this issue Mar 11, 2021 · 16 comments
Assignees
Labels

Comments

@matentzn
Copy link
Collaborator

The logical definitions for ENVO_01001479 and ENVO_01001784 are equivalent. To replicate the test (also useful for qc!)

robot reason -I http://purl.obolibrary.org/obo/envo.owl --equivalent-classes-allowed asserted-only -o reason.owl
@pbuttigieg pbuttigieg added the bug label Mar 11, 2021
@pbuttigieg pbuttigieg self-assigned this Mar 11, 2021
@pbuttigieg
Copy link
Member

@matentzn many thanks - looking in to it

@cmungall
Copy link
Member

cmungall commented Apr 8, 2021

The text definition of ENVO_01001784 compound astronomical body part is very different from the equivalence axiom. I recommend the equivalence axiom is removed

@matentzn
Copy link
Collaborator Author

Please, someone, prioritise this, it is causing me big problems in many parts of the OBO sphere:

  • Add check for logical equivalence with robot reason --equivalent-classes-allowed asserted-only

@matentzn
Copy link
Collaborator Author

cc @kaiiam @wdduncan

@kaiiam
Copy link
Contributor

kaiiam commented Mar 14, 2022

@matentzn Their will be an ESIP semantic harmonization meeting wednesday this week where we'll be looking at #1172. I'll include fixing ENVO_01001784 (which @cmungall suggests to remove the EQ axiom) comparing against ENVO_01001479.

@kaiiam kaiiam mentioned this issue Mar 14, 2022
@kaiiam
Copy link
Contributor

kaiiam commented Mar 16, 2022

@matentzn see 86014b1 should fix this issue.

@wdduncan
Copy link
Member

@kaiiam Thanks for fixing. It is hard for me to tell what the fix was just looking at the diff file though :(

@matentzn
Copy link
Collaborator Author

I agree!

@kaiiam
Copy link
Contributor

kaiiam commented Mar 31, 2022

Ya not sure why the commit is now showing the whole set of changes not just the one commit possible because it was merged? Regardless we changed the EQ axiom so they aren't duplicated anymore.

@matentzn
Copy link
Collaborator Author

matentzn commented May 4, 2022

ping @kaiiam @pbuttigieg I am introducing so many special pipelines for processing ENVO, would be great if we could priotise tighter release cycles :)

@kaiiam
Copy link
Contributor

kaiiam commented May 4, 2022

I agree ENVO is overdue for a release. Regarding making them more frequently I agree. At the moment, I think not having full SOP is holding us back. We should prioritize making that happen. I made most of an ENVO release SOP https://github.com/EnvironmentOntology/envo/wiki/Making-a-release but we never finished it we're missing the last step with a release artifact generating script to create the MD file with new terms etc.

I'm in the last stages of writing my dissertation at the moment so if anyone else @cmungall @pbuttigieg @wdduncan @ddooley @turbomam has time to do this now great if not I'll give it a shot once I have time again.

@matentzn
Copy link
Collaborator Author

matentzn commented May 4, 2022

I have 0 problems to do this, its more the socio-technical issues I am worried about. If you can decide among yourselves that "if it is on master, its ready for publication", I can just create releases all the time, every 2 months if needed. I just don't want to go through the debate of whether "some things need to be changed" before the release every time, if you know what I mean :P

@kaiiam
Copy link
Contributor

kaiiam commented May 4, 2022

Fair points @matentzn we're hoping to have an ENVO governance meeting Jun 6, 2022 15:00 UTC. I think making a decision about release cycles should be on the agenda.

@matentzn
Copy link
Collaborator Author

Consider a @pbuttigieg independent release process (not review process!) a top priority on that governance call! These ENVO format errors are really taking up too much of my time.

@kaiiam
Copy link
Contributor

kaiiam commented May 26, 2022

Sorry to hear that ENVO issues are eating up a lot of your time @matentzn. I think an independent release process that could be triggered automatically seems pretty reasonable.

@matentzn
Copy link
Collaborator Author

No problem at all :) Just want to make sure it will actually happen so we can push hotfixes whenever necessary like we do with other ontologies!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants