search

semantic-systems / NLIWOD

Collection of tools, utilities, datasets and approaches towards realising natural language interfaces for the Web of Data.
GNU Affero General Public License v3.0
94 stars 32 forks source link

Merge SPARQL functionality #65

Open LukasBluebaum opened 5 years ago

LukasBluebaum commented 5 years ago

In qa.commons, there are 2 different ways to fire queries to an endpoint, SPARQL and SPARQLExecutor. This becomes tedious to keep clean, especially when both share the same (hardcopied) code. Consider merging the functionality in favor of SPARQL.

RicardoUsbeck commented 5 years ago

Makes sense. @Aklakan as an expert, do you have any recommendation which to keep?

LukasBluebaum commented 5 years ago

There was already a TODO in the code that SPARQL should be kept. I guess thats because it also supports caching and the other one doesn't.