Skip to content

case_sparql_select: Add --use-prefixes flag - #86

Merged
ajnelson-nist merged 3 commits into
developfrom
case_sparql_select_add_use_prefixes
Dec 6, 2022
Merged

case_sparql_select: Add --use-prefixes flag#86
ajnelson-nist merged 3 commits into
developfrom
case_sparql_select_add_use_prefixes

Conversation

@ajnelson-nist

@ajnelson-nistajnelson-nist commented Dec 6, 2022

Copy link
Copy Markdown
Member

This patch series adds an option to use the rdflib.NamespaceManager to apply prefixes in the input graph to all rdflib.URIRefs emitted in the ouput table. A unit test is added to confirm expected output is generated.

A follow-on patch will regenerate Make-managed files.
Signed-off-by: Alex Nelson <alexander.nelson@nist.gov>
Signed-off-by: Alex Nelson <alexander.nelson@nist.gov>
@ajnelson-nistajnelson-nist added this to the 0.9.0 milestone Dec 6, 2022
@ajnelson-nist
ajnelson-nist marked this pull request as ready for review December 6, 2022 15:10
@ajnelson-nistajnelson-nist changed the title Add and test --use-prefixes flag for case_sparql_selectcase_sparql_select: Add --use-prefixes flagDec 6, 2022
Signed-off-by: Alex Nelson <alexander.nelson@nist.gov>
@ajnelson-nist

Copy link
Copy Markdown
MemberAuthor

I pushed a patch after a local test confirmed no ill effects. @kchason , if you have no objection, I'll merge once CI passes.

@ajnelson-nist
ajnelson-nist merged commit 10589e7 into developDec 6, 2022
@ajnelson-nist
ajnelson-nist deleted the case_sparql_select_add_use_prefixes branch December 6, 2022 16:22
Sign up for freeto 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

@ajnelson-nist@kchason