-
Notifications
You must be signed in to change notification settings - Fork 23
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
Weird wording in spec of Iri Compaction #75
Comments
The same also applies for 2.8.2 and 2.8.5 |
Order matters, after running through the steps, |
Agree with @gkellogg. Propose close, not an issue :) |
From WG call: @gkellogg to consider whether additional text could be valuable to explain how to read the algorithm steps. |
Consider informative notification for how the container matching works. |
This issue was discussed in a meeting.
View the transcriptRob Sanderson: link: #75Rob Sanderson: first one seems like a misunderstanding about the algorithm. Greg clarified in the issue. no issue and we can close Gregg Kellogg: maybe more descriptive overview could be helpful Ivan Herman: the comment came in 8 days ago and the response came in 8 days ago as well. We should probably wait 1 week when we have issue from external members. Rob Sanderson: close or make note about editorial changes? Adam Soroka: +1 to editorial. Gregg Kellogg: make it editorial and look to make sure the description is more robust Ivan Herman: marked as editorial and will add reference to the notes Rob Sanderson: we can close without further discussion Proposed resolution: Mark #75 as editorial, for Gregg to consider whether additional text is needed to explain how to read the algorithm steps (Rob Sanderson) Gregg Kellogg: +1 Rob Sanderson: +1 Jeff Mixter: +1 Ivan Herman: +1 Adam Soroka: +1 Benjamin Young: +1 Tim Cole: +1 Resolution #2: Mark #75 as editorial, for Gregg to consider whether additional text is needed to explain how to read the algorithm steps |
…lues of _containers_. Fixes w3c/json-ld-syntax#75.
The spec of iri compaction states in 2.8
In other words, this boils down to always adding
@graph@index
and@graph@index@set
to thecontainers
array.So either the spec can be simplified or some copy-paste error occurred. Could you please clarify?
The text was updated successfully, but these errors were encountered: