Skip to content

[WIP] Implements SPARQL-level config for ASK queries#2714

Draft
egonw wants to merge 1 commit into
mainfrom
update/askConfigurableToo
Draft

[WIP] Implements SPARQL-level config for ASK queries#2714
egonw wants to merge 1 commit into
mainfrom
update/askConfigurableToo

Conversation

@egonw

@egonw egonw commented Aug 23, 2025

Copy link
Copy Markdown
Collaborator

Description

So far, ASK queries were always run against the globally defined SPARQL endpoint. This patch added the option to configure this for each SPARQL query separately.

Caveats

Non-trivial to test.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
    • I have made corresponding changes to the documentation
  • This change requires new dependencies (please list)

Testing

For the example configured ASK query:

image

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have not used code from external sources without attribution
  • I have considered accessibility in my implementation
  • There are no remaining debug statements (print, console.log, ...)

@egonw egonw self-assigned this Aug 23, 2025
@egonw

egonw commented Aug 23, 2025

Copy link
Copy Markdown
Collaborator Author

Locally, I do not get the error. But it's reproducible. Just not sure right now how to fix this. Will try that later.

@fnielsen

Copy link
Copy Markdown
Collaborator

There is an error for test_work_cito. Is this just a temporary backend error? Or is it a Scholia error?

@egonw

egonw commented Aug 25, 2025

Copy link
Copy Markdown
Collaborator Author

There is an error for test_work_cito. Is this just a temporary backend error? Or is it a Scholia error?

@fnielsen, I have yet to explore this. That's why I tagged this PR as draft. I am sorry that was not clear enough. I will debug asap, but in a meeting today.

@fnielsen

Copy link
Copy Markdown
Collaborator

Ok. I did not see the "draft".

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