-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/blazegraph #521
Feature/blazegraph #521
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kudos for getting it to work! Minor comments below.
Anything I need to know about installing Blazegraph before I try this locally (that isn't already in the Blazegraph manual)?
Apologies for the oversight on auto-formatting. I set it to 'only changed code' a while ago and got lazy checking changes before staging. Follow the blazegraph quickstart and add namespaces |
Quick update on testing at my end:
|
Another update: I successfully installed PostgreSQL 14 and migrated my data (which was a bit of a hassle). Most backend tests pass now, except for the new In the first assertion that I'll hold off on further testing until this is resolved. |
Did not realise this. I use PostgreSQL 13, because this is what most of our servers run. Good to realise that this can have an effect on the code.
Thanks for clearing this up. Turns out SPARQLStore keeps a rolling dictionary of nsBinding, I suspect all common namespaces occurring in triples in the store. I only ran the test in isolation. I'll adapt the test. |
Great
This is unfortunate. As we discussed, this is best solved by installing a specific version in the deployment script. I'll make sure the documentation mentions this. |
I removed the |
I fixed this issue. |
A few things going on in this PR:
https
for schema.org, frontend useshttp
(sigh)To test this, you will need a virtual environment with Python 3.8.x (tested on 3.8.9). If ontology classes do not show up in the picker, clear the ontology graph and re-run
rdfmigrate
.I'm working on the updated documentation.
Closes #520, closes #476