Skip to content

[WDQS] PREFIX not aligned with concept URI #798

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.

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

Closed
fdeuze opened this issue Nov 5, 2024 · 2 comments
Closed

[WDQS] PREFIX not aligned with concept URI #798

fdeuze opened this issue Nov 5, 2024 · 2 comments

Comments

@fdeuze
Copy link

fdeuze commented Nov 5, 2024

Setup:
Using WDQS:2 (wdqs0.3.142 from Oct 28, 2024) which solve correctly the issue to set the WIKIBASE_CONCEPT_URI required in wdqs-updater container (Note: However fix is needed in docker-compose.xml to set the correct version here)

Issue:
The PREFIX in SPARQL query is not using the correct concept URI in wdqs container.
Indeed, setting the log "com.bigdata.rdf.sail.webapp.BigdataRDFContext" to level "debug", messages show:
PREFIX wdt: http://wikibase/prop/direct/
PREFIX wd: http://wikibase/entity/

instead of something like:

PREFIX wdt: https://concept_uri/prop/direct/
PREFIX wd: https://concept_uri/entity/

Recreation:
Any SPARQL query using prefix like wd or wdt from wdqs-fontend container UI.
For example this query doesn't return anything:
select *
where
{
?s ?p wd:Q1
}
LIMIT 10

whereas this query return the expected result:
select *
where
{
?s ?p https://concept_uri/entity/Q1
}
LIMIT 10

@rti
Copy link
Contributor

rti commented Nov 8, 2024

Hi,

Thanks a lot for reaching out!

However fix is needed in docker-compose.xml to set the correct version here

Thanks for pointing that out. We released this as a fix to deploy yesterday https://github.com/wmde/wikibase-release-pipeline/releases/tag/deploy%403.0.3

We will have a look into the prefixes issue!

rti added a commit that referenced this issue Nov 28, 2024
As reported in #798, the "standard prefixes" injected by WDQS do not align with the WIKIBASE_CONCEPT_URI setting introduced in #771, nor do they match the corresponding values for Wikidata. Currently, the WDQS-injected "standard prefixes" (such as wd:) reference http://wikidata/, which is the internal hostname of the wikibase/mediawiki container within the Docker network.

Particularly in the context of federation, it is essential to maintain the "standard prefixes" for referencing Wikidata [1] [2]. To accommodate the local Wikibase instance, prefixes can be set inline, as is currently done on wikibase.cloud [2]. This pull request addresses this and updates the "standard prefixes" to point to Wikidata. Additional context can be found in [3].

[1] https://www.mediawiki.org/wiki/Wikibase/Wikibase.cloud/First_steps#View_your_data_using_the_Query_Service
[2] https://phabricator.wikimedia.org/T335448
[3] https://phabricator.wikimedia.org/T379232
@rti
Copy link
Contributor

rti commented Nov 28, 2024

Closing this as the PR got merged to main

@rti rti closed this as completed Nov 28, 2024
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

No branches or pull requests

2 participants