Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@egonw
Copy link
Member

@egonw egonw commented May 16, 2022

Maybe solves #862 without upgradeing to the latest Jena (see #748)

@egonw egonw requested a review from johnmay May 16, 2022 06:09
@johnmay
Copy link
Member

johnmay commented May 16, 2022

Does not solve, I don't know too much about IORDF why do we need it, what advantages does the dependency bring?

@egonw
Copy link
Member Author

egonw commented May 16, 2022

I don't know too much about IORDF why do we need it, what advantages does the dependency bring?

I am not comfortable with this question. If we ask that about any functionality, we get very long discussions. For what purpose? CDK was designed as library of cheminformatics functionality, not to be a project to decide what is important (for someone) and what is not.

@johnmay
Copy link
Member

johnmay commented May 16, 2022

My point was for the functionality of OWL you need it would perhaps be shorter just to write your own xml input/output

@johnmay
Copy link
Member

johnmay commented May 16, 2022

See #748 should be good now once build is rounded up

@egonw
Copy link
Member Author

egonw commented Jun 5, 2022

No longer relevant.

@egonw egonw closed this Jun 5, 2022
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