Skip to content

Implement the RDFJS Data Model spec.#142

Merged
dmitrizagidulin merged 14 commits intomasterfrom
dz_rdfjs_data_model
Aug 22, 2016
Merged

Implement the RDFJS Data Model spec.#142
dmitrizagidulin merged 14 commits intomasterfrom
dz_rdfjs_data_model

Conversation

@dmitrizagidulin
Copy link
Copy Markdown
Contributor

@dmitrizagidulin dmitrizagidulin commented Aug 16, 2016 •
edited
Loading

Partly implements issue #137.

  • Implement DefaultGraph
  • Implement literal.hasLanguage() as a serialization helper
  • Store original Variable name in .value
  • Literal - only serialize datatype if non-default
  • Use duck typing to check for Node instance
  • Update termType usage to match RDFJS interface

This was referenced Aug 16, 2016
Open
Merged
Merged
Merged
@dmitrizagidulin
Copy link
Copy Markdown
Contributor Author

dmitrizagidulin commented Aug 16, 2016

Corresponding downstream PRs (to change the termType values):

@nicola
Copy link
Copy Markdown
Member

nicola commented Aug 17, 2016

AWESOME

@dmitrizagidulin
Copy link
Copy Markdown
Contributor Author

dmitrizagidulin commented Aug 17, 2016

@timbl Ready for review.

@dmitrizagidulin
Copy link
Copy Markdown
Contributor Author

dmitrizagidulin commented Aug 18, 2016

@timbl any other corrections to this PR, or can we merge it?

@dmitrizagidulin dmitrizagidulin merged commit 9fb2260 into master Aug 22, 2016
@dmitrizagidulin dmitrizagidulin deleted the dz_rdfjs_data_model branch August 22, 2016 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dmitrizagidulin @nicola @deiu