-
Notifications
You must be signed in to change notification settings - Fork 30
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
CI failure, maybe due to overlapping synonyms #3084
Comments
looks like the check went from case sensitive to case insensitive. This is
probably a good thing but your PR should not have been caught in the
crossfire. If someone is able to make the fix (here or another PR), I would
demote the TAO derived abbrevs
…On Sun, Oct 1, 2023 at 3:58 AM Charles Tapley Hoyt ***@***.***> wrote:
In my recent PRs #3082 <#3082>
and #3083 <#3083>, there
appear to be some QA check failures occurring. From how they look, it's
because different terms are sharing exact synonyms:
entity,property,valuehttp://purl.obolibrary.org/obo/UBERON_0002633,http://www.geneontology.org/formats/oboInOwl#hasExactSynonym,nVhttp://purl.obolibrary.org/obo/UBERON_2000438,http://www.geneontology.org/formats/oboInOwl#hasExactSynonym,phyhttp://purl.obolibrary.org/obo/UBERON_2005093,http://www.geneontology.org/formats/oboInOwl#hasExactSynonym,NVhttp://purl.obolibrary.org/obo/UBERON_8440028,http://www.geneontology.org/formats/oboInOwl#hasExactSynonym,PHY
This wasn't an issue in the last merge, so I'm guessing it has to do with
updates to odk/robot. Can someone take a look at these and solve them?
—
Reply to this email directly, view it on GitHub
<#3084>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAMMOJAC3JOXWZO6OZMNMDX5FEGFANCNFSM6AAAAAA5OHTJIY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
That's unexpected. The ODK version in the QC is v1.4.1. |
Updating the ODK to v1.4.3 with new OWLAPI and ROBOT versions in the QC will fix the issue. |
It appears that this was not fixed by the ODK update. #3082 is still failing |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In my recent PRs #3082 and #3083, there appear to be some QA check failures occurring. From how they look, it's because different terms are sharing exact synonyms:
This wasn't an issue in the last merge, so I'm guessing it has to do with updates to odk/robot. Can someone take a look at these and solve them?
The text was updated successfully, but these errors were encountered: