Skip to content
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

Optimise vocab loading #133

Closed
wu-lee opened this issue Nov 17, 2021 · 3 comments
Closed

Optimise vocab loading #133

wu-lee opened this issue Nov 17, 2021 · 3 comments

Comments

@wu-lee
Copy link
Contributor

wu-lee commented Nov 17, 2021

Generating the vocabs gets very slow when ther are more than one, delaying the map loading for 30 seconds or longer. This can probably be sped up with more selective loading and/or caching.

@wu-lee
Copy link
Contributor Author

wu-lee commented Nov 25, 2021

It turns out this is just that vocab queries don't have the default graph uri set.

I compared the speed of get_vocabs.php to a command-line SPARQL query via curl (see seodo using the same query, and the latter was less than half a second. The critical difference was the default-graph-uri.

This info also is in the config/*/default-graph-uri.txt files of a map website, so in principle get_vocabs.php could get it from there, but there vocab queries are done once and currently shared by all datasets, so it isn't clear how to resolve that and pick the right one.

Therefore I decided it was a parameter like endpoint in config.json, which needs to be provided once per vocab query. (I've left it optional however, so existing sea-map sites will still work without it.)

@wu-lee
Copy link
Contributor Author

wu-lee commented Nov 25, 2021

I've also updated ica-project to use the new config.json setting. Will deploy this along with all the other ICA updates shortly.

@ColmMassey
Copy link
Contributor

Loading times back to normal.

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