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

Add oaklib (libraries in path and runoak CLI) to ODK #584

Closed
kltm opened this issue Apr 21, 2022 · 7 comments · Fixed by #586
Closed

Add oaklib (libraries in path and runoak CLI) to ODK #584

kltm opened this issue Apr 21, 2022 · 7 comments · Fixed by #586
Assignees

Comments

@kltm
Copy link
Contributor

kltm commented Apr 21, 2022

This is a request to add OAK (ontology access kit) to the ODK.

https://github.com/INCATools/ontology-access-kit

Tagging @cmungall

@matentzn
Copy link
Contributor

@gouttegd
Copy link
Contributor

It is on PyPI (despite what the README says) under the name oaklib, and it does not seem to have any non-Python dependencies, so yes, it shouldn’t require anything more than updating that file.

@matentzn
Copy link
Contributor

Thank you @gouttegd :)

The tool does so many things, not sure if it has some undocumented dependencies, like dot or similar.

@gouttegd
Copy link
Contributor

Argh, the viz command requires obographviz which requires Node.js!

(Runs away in fear)

@cmungall
Copy link
Contributor

cmungall commented Apr 21, 2022 via email

@gouttegd
Copy link
Contributor

Only for creating graph images, if I understand correctly?

But still, that may be useful and after all one point of the ODK is to provide the tools ontology editors/engineers might need, without asking them to deal with Python or Node packages.

I did a quick check, apparently adding Node.js, NPM, and the Node packages required by obographviz would add ~100 MB to the ODK. If there’s a need for OAK’s viz command, I’d be in favour of adding that to odkfull.

@cmungall
Copy link
Contributor

cmungall commented Apr 21, 2022 via email

@gouttegd gouttegd self-assigned this Apr 21, 2022
@gouttegd gouttegd mentioned this issue Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants