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

Dumps #282

Merged
merged 13 commits into from
Apr 26, 2020
Merged

Dumps #282

merged 13 commits into from
Apr 26, 2020

Conversation

maxlath
Copy link
Member

@maxlath maxlath commented Mar 13, 2019

entities dump

add schema:dateCreated (from entity.created)
add schema:dateModified (from entity.updated)
add schema:version (from entity._rev)
add wikibase:statements (statements count)
add wikibase:labels (labels count)
remove skos:prefLabel (duplicated with rdfs:label)

items dump

sketched but commented-out as it doesn't pass the validation and might not be a good idea after all (see privacy concerns)

@maxlath maxlath requested a review from jum-s March 13, 2019 16:47
echo "
@prefix wdt: <http://www.wikidata.org/prop/direct/> .
@prefix inv: <http://inventaire.io/entity/> .
@prefix invexamplar: <http://inventaire.io/items/> .
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wuhu what a brand new name ha ?!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe invex as its should stay a short string as a prefix ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah that was temporary, and with the whole items dump being commented-out now, that could even be removed

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed in c556bb8


# We only need the items entities for now
curl -s "${dbHost}/items-prod/_design/items/_view/byEntity" |
jq '.rows[] | [.id, .key[0]] | join(" wdt:P1574 ")' |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

arent you pushing item ID in the ttl with this line ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, that's why it's commented-out in prepare_dumps, but maybe that should just be removed(?) Keeping track of it somewhere could be of value though

Copy link
Contributor

@jum-s jum-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approving since, according to max, the rebase is only a translation from coffee to es6

@maxlath maxlath merged commit c263e1a into master Apr 26, 2020
@maxlath maxlath deleted the dumps branch April 26, 2020 12:29
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

Successfully merging this pull request may close these issues.

2 participants