-
Notifications
You must be signed in to change notification settings - Fork 291
Wikidata integration + Refactoring Core to accept new formats #155
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
Conversation
various changes
…xtractor[WikiPage]
final Live merge
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.
Naming coherence:
val WikidataNamespaceSameAs = new Dataset("wikidata-namespace-sameas")
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.
Should use WikiPage.sourceIri (which does not exist).
WikiPage has a revision property though, so a valid (and coherent with current framework) sourceIri could be:
WikiPage.title.pageIri + "?oldid=" + revision
similar to PageNode.sourceUri
PageNode.title.pageIri does not give enough context to this quad.
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.
Remove space after WikiPage
fix review comments
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.
Is this method doing anything?
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.
no it's the dummy Wikidata Extractor Structure that Dimitris created in the beginning of the task , we can remove the whole file it's obsolete anyways
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.
go ahead and remove it Hady
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.
done
…ata.dbpedia.org/resource/XXX
|
last edits : 2- wikidata uris to be : "http://www.wikidata.dbpedia.org/resource/" instead of ""http://www.wikidata.org/entity/" |
Wikidata integration + Refactoring Core to accept new formats
Wikidata integration + Refactoring Core to accept new formats
This pull request is mainly the output of GSoC2013 Wikidata Integration inside DBpedia Project
there are two main Issues left to be done after the merge :
1 - there are two warnings left for JsonLift: 'withFilter' method does not yet exist on net.liftweb.json.JsonAST.JValue, using `filter' method instead, try mvn clean install to see them
2- the Design of the Wikidata Extractors and JsonWikiParser , were temporary design to cope with the old core , now the new core is redesigned so what we should do :