Skip to content

Conversation

@hlapp
Copy link
Member

@hlapp hlapp commented Nov 11, 2025

No description provided.

@hlapp hlapp requested a review from gaurav November 11, 2025 22:38
@hlapp
Copy link
Member Author

hlapp commented Nov 11, 2025

@gaurav please check, perhaps some of these updates (such as for phyx.js) are premature given what the code actually does right now?

Copy link
Member

@gaurav gaurav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I've recapitalized "PHYX" to "Phyx" because that's how we capitalize it in the paper.

The README will need a whole lot of rewriting, but we might also need to change some of the other elements of this pipeline, e.g. we now prefer to write out OWL ontologies in n-Quads directly in phyx.js rather than writing out JSON-LD and then using another tool to convert that to RDF (in this repo, rdfpipe). So all of this might need to change eventually. But just linking to the paper and the other repos are an essential first step.

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.

3 participants