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

ODK 1.4 migration #12

Merged
merged 3 commits into from
Mar 1, 2023
Merged

ODK 1.4 migration #12

merged 3 commits into from
Mar 1, 2023

Conversation

matentzn
Copy link
Collaborator

This PR contains a lot of updates from ODK 1.4 migration. If anything goes wrong (which it shouldn't, but because Dron is big I cant exclude the possibility), I will fix it!

@hoganwr
Copy link
Collaborator

hoganwr commented Feb 20, 2023

The build process is getting stuck. It won't go past here:

dron.Makefile:24: warning: overriding recipe for target 'dron-lite.owl'
Makefile:627: warning: ignoring old recipe for target 'dron-lite.owl'
echo "ODK Makefile version: v1.4 (this is the version of the ODK with which this Makefile was generated,
not the version of the ODK you are running)" &&
echo "ROBOT version (ODK): " && robot --catalog catalog-v001.xml --version
ODK Makefile version: v1.4 (this is the version of the ODK with which this Makefile was generated, not the version of the ODK you are running)
ROBOT version (ODK):
ROBOT version 1.9.3
robot --catalog catalog-v001.xml convert --input dron-edit.owl --format ofn --output tmp/dron-preprocess.owl
robot --catalog catalog-v001.xml reason --input tmp/dron-preprocess.owl --reasoner ELK --equivalent-classes-allowed asserted-only
--exclude-tautologies structural --output test.owl && rm test.owl

@matentzn
Copy link
Collaborator Author

On the server, or on the local machine? How large is the servers available main memory? This will be the deciding factor here!

@matentzn
Copy link
Collaborator Author

(should be at least 32GB)

@hoganwr
Copy link
Collaborator

hoganwr commented Feb 20, 2023 via email

@hoganwr
Copy link
Collaborator

hoganwr commented Feb 20, 2023 via email

@hoganwr
Copy link
Collaborator

hoganwr commented Feb 20, 2023

Ok I got the process to run to completion.

Problem: the ChEBI import is broken. All the chebi classes show up at the top level under owl:Thing.

@matentzn
Copy link
Collaborator Author

Anything in the log files to go on?

@hoganwr
Copy link
Collaborator

hoganwr commented Mar 1, 2023

Sorry for taking so long to respond. I think the issue was that I tested it out on the main branch instead of a release branch. I've just run the process again and the problem did not recur. The only difference being this time I ran it on a release branch. It looks good. I will merge the pull request

@hoganwr hoganwr merged commit b7b27a8 into main Mar 1, 2023
@hoganwr hoganwr deleted the odk14 branch March 1, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants