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.

Sign up for GitHub

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

Jump to bottom

Streamline URI and CURIE parsing in the reader #246

Merged

cthoyt merged 3 commits into main from update-parse

Nov 25, 2024

Member

cthoyt commented Nov 25, 2024

Rename Reference.from_curie to Reference.from_curie_to_uri. This has the benefit that it can be typed differently, and also reflects its different functionality from the base class.
Updated node iteration to directly construct references. This reduces some superfluous code in indexing them in a mapping
Pass ontology_prefix through to functions where it was missed

cthoyt added 3 commits

November 25, 2024 10:46


          Streamline parsing from curies

e95ead4


          Rename function

c6c09c9


          Simplify node iteration

d3c9752

cthoyt merged commit c683915 into main

11 checks passed

cthoyt deleted the update-parse branch

November 25, 2024 10:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet