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

Rework initial explanation of shape tree structure and model #44

Merged
merged 13 commits into from
Jun 3, 2021

Conversation

justinwb
Copy link
Contributor

@justinwb justinwb commented May 3, 2021

@justinwb justinwb requested a review from ericprud May 28, 2021 21:06
@justinwb justinwb marked this pull request as ready for review May 28, 2021 21:06
index.bs Outdated Show resolved Hide resolved
index.bs Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
</tr>
</tbody>
</table>

Copy link
Contributor

Choose a reason for hiding this comment

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

PROPOSE:

value definition
st:ShapeTreeContainer a server-managed Resource that enumerates nested resources
st:ShapeTreeResource a user-managed RDF resource
st:ShapeTreeNonRDFResource a user-managed non-RDF resource (e.g. binary or image)

I guess this could go somewhere as well:

Resource R is nested within a Container C if a relative URL of '..' evaluated against a base of R produces C.

Copy link
Contributor Author

@justinwb justinwb May 31, 2021

Choose a reason for hiding this comment

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

a server-managed Resource that enumerates nested resources

A container resource isn't completely server-managed though, (primarily) just the containment triples.

index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Show resolved Hide resolved
@ericprud
Copy link
Contributor

ericprud commented Jun 1, 2021

for parallel-ism, propose s/st:hasShapeTreeInstanceRoot/st:hasRootShapeTreeInstance/

@justinwb
Copy link
Contributor Author

justinwb commented Jun 1, 2021

for parallel-ism, propose s/st:hasShapeTreeInstanceRoot/st:hasRootShapeTreeInstance/

Okay - made this change under the operations refactor branch to save another branch merge - 03e63e9

@justinwb justinwb merged commit 076376c into main Jun 3, 2021
@justinwb justinwb deleted the shape-trees-model branch June 3, 2021 17:30
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