-
Notifications
You must be signed in to change notification settings - Fork 18
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
Create ECTO release using axioms.owl mappings #99
Comments
Whats the timeline on this? |
I mean, whats the deadline? |
I think they want this as soon as you can do it :( |
OK. |
Bill, what format do they want this in? OWL, OBO? Is there any document/issue that describes what exactly they want? |
Looks like you have been matching against an old version of ECTO... Can you re-run against the current release? None of the mappings match to anything actually in ECTO right now:
There is no |
Here are instructions to merge your axioms into the ECTO release:
This will generate the OWL and OBO versions for ECTO plus your mappings. Let me know when |
BTW: the corresponding ROBOT snippets are:
|
There are also only a few mappings IMO. 35? Werent there many more before? |
@matentzn Sorry a little confused.
Does |
Yeah that is the problem.. That file is from the neolithic area.. Change to this:
|
Yeah looks better! After you merged it, can you run:
And inspect
Who can answer these questions? |
@diatomsRcool can you answer @matentzn questions? For now, I think we should exclude the boomer mappings. When I try to run:
I receive the error:
|
Well, does the file exist? |
No. I removed it. But I don't know why run.sh would expect it. |
I just added it here. You can change that in the file to:
Let me know if that works! |
I fixed some problems boomer was having with the input owl files. I will make a separate ticket for those. The command:
runs successfully. |
What is the status of this? |
Please create a new release of ECTO using the mappings in src/mappings/axioms.owl.
There is also a src/mappings/axioms-boomer.owl file created using boomer. I don't know if you want to incorporate it.
cc @diatomsRcool @cmungall @wdduncan
The text was updated successfully, but these errors were encountered: