Update of the main branch to work with QLever
#2715
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add ProxyFix middleware to handle proxy headers correctly - Configure runserver.py to bind to host 'tajo' - Fixes CSS/static asset paths when deployed behind proxy 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Collaborator
|
I wonder if the proxy info can be integrated in the configuration system we have, instead of having it hardcoded. |
Collaborator
|
Yes, the hardcoding
Yes, or perhaps the hardcoded variable could be an input variable? |
1. Explicitly ask for application/sparql-results+xml (default for Blazegraph) 2. Show QLever exception when available 3. Hack lables by using `REPLACE(STR(?var), ".*/", "") AS ?varLabel` 4. Remove Blazegraph-specific query hints (hint:Query, hint:Prior) 5. Add missing prefix for ASK query and statistics query
1. Add missing PREFIX geof: ... definition 2. Improve error message 3. Hack another SERVICE wikibase:label
1. Automatic pre-formatting using qlue-ls and sed 2. Manual post-correction of all queries (puuh)
added missing prefix
Added OWL prefix to the SPARQL query for better ontology support.
…st-of-authors.sparql
…ia, qlever branch
Collaborator
|
@fnielsen, I think the branch is in fair shape. I do not have enough experience with Codacy to get the 5 issues fixed. They seem harmless to me (the Markdown ones I do not even get the problem of...). Anything you like to see improved? |
… Adriano for getting this started
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@fnielsen, this is @egonw typing here, this PR started out as "Configure Flask app for proxy deployment" but now has many updates to make Scholia work with QLever. The live instance is shaping up really well. But there is so much here, plz ping me to talk you through the work.
@hannahbast, sorry for updating the title and this description. Since everything was merged into the
qleverbranch, it all shows up on this PR.