Conversation
|
C.1.1 Packaging and initial storage
|
maximelefrancois86
left a comment
There was a problem hiding this comment.
broken Fig 52 ?
Both. This is a untidy corner of both schema.org and gs1 product. In Page 115 of the GS1 Digital Link Standard, it must be a class because of the relationships between lots / serial numbers. It's an instance of schema:IndividualProduct in schema.org land. The problem is also reviewed in gs1/WebVoc#38. I'm comfortable leaving it as both in this particular example, I added a schema:IndividualProduct type to make the distinction clear. Commercial implementation (me) run into this issue every day. Do you think that a short explanatory blurb would be beneficial in the long example?
Fixed.
A version of this is done in "C.1.4.1 Recording data and sensor tasking through sensor deployments". |
It's rendering properly locally here. What breakage are you seeing? |
ldesousa
left a comment
There was a problem hiding this comment.
Besides the individual comments there are two broad aspects to consider:
- Use of predicates: better to stick to the intended use.
- TTL documents: each should be a contained asset, including all individual declarations necessary. In alternative encapsulate recurrent individuals (e.g. sensors) in their own document, to be then "imported" with
@prefix.
|
To be moved to a markdown example. |
|
...it was the night before Christmas and all through the repo, nothing was stirring, not even an issue... |
|
@oldskeptic to tighten up the section at the top of the md document to more strongly draw attention to punning, merging OWL with schema.org and GS1 concerns. |
ldesousa
left a comment
There was a problem hiding this comment.
I only request changes for the various typos in the Markdown documents. Otherwise, I am fine with merging, even if none of my comments on the RDF are addressed.
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
added link to IBS-TH2 description in Complex Systems pattern; added sosa:startTime and sosa:endTIme to deployments
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
Co-authored-by: Luís de Sousa <luis.de.sousa@protonmail.ch>
dr-shorthair
left a comment
There was a problem hiding this comment.
A few minor issues from @ldesousa should still be considered.
|
@oldskeptic a few small outstanding questions from Luis. @sgrellet @KathiSchleidt @rob-metalinkage @ldesousa could you look at this pr please? Needs one review. |
My comments from last week are now outdated. Meanwhile Ted reviewed the text. I am happy if you merge. |
Thanks @ldesousa - can you review/approve please? |
This refactors the long ACME Brewery according to all the changes in SOSA and follows the new file layout.