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

Use of undefined terms "graph-based" and "tree-based" #161

Closed
selfissued opened this issue Jan 29, 2020 · 3 comments · Fixed by #278
Closed

Use of undefined terms "graph-based" and "tree-based" #161

selfissued opened this issue Jan 29, 2020 · 3 comments · Fixed by #278
Assignees
Labels
Editorial Editors should update the spec then close PR exists There is an open PR to address this issue

Comments

@selfissued
Copy link
Contributor

The specification has numerous uses of the undefined terms "graph-based" and "tree-based". They are apparently used to attempt to define semantic properties of DIDs, but that goal isn't accomplished by using terms that are not commonly understood.

Example uses of these terms include:

  • "These documents are graph-based data structures that are typically expressed using [JSON-LD], but can be expressed using other compatible graph-based data formats."

  • "Implementations need not rely on graph-based processing of DID documents to locate metadata contained in the DID document when the DID includes a DID fragment. Tree-based processing can be used instead."

  • "Modelling of complex multi-entity relationships is provided through the use of a graph-based data model."

Please either define these terms and say why they are relevant to DIDs and their semantics or remove their uses from the specification.

@rhiaro rhiaro added the Editorial Editors should update the spec then close label Feb 4, 2020
@selfissued
Copy link
Contributor Author

I will write a PR removing the use of these undefined terms.

@selfissued
Copy link
Contributor Author

These terms do not provide actionable information. They are more philosophy than normative statements. Therefore, their usage should be removed.

@kdenhartog
Copy link
Member

Given our current approach of supporting lossless conversion, I think it's acceptable to move away from these descriptions as well. The underlying data model shouldn't play as large of a role anymore if we get lossless conversion correct.

@peacekeeper peacekeeper added the PR exists There is an open PR to address this issue label May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Editorial Editors should update the spec then close PR exists There is an open PR to address this issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants