-
Notifications
You must be signed in to change notification settings - Fork 47
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
[NEW model] early warning notification by Andreas & Timothy patch 1 #484
Conversation
New model for early warning notification added
This file contains the new aspect structure for the early warning notification aspect.
@Timothy-Ra Did you create any Github issue for this Pull Request? If not, then please first create an issue here.After that we go for approval process. |
updated model with respect to the MS2 criteria
add sample json content
@@ -0,0 +1,38 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should only contain { "status" : "release"}
. please delete the content of this file.
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> . | ||
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> . | ||
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> . | ||
@prefix : <urn:samm:org.eclipse.examples:1.0.0#> . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please replace the org.eclipse.examples
with io.catenax.early_warning_notification
|
||
:QualityAppFilter a samm:Entity ; | ||
samm:preferredName "Affected Population FIlter"@en ; | ||
samm:description "A set of filters and/or a list definieng the affected population"@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo definieng
:populationFilter a samm:Property ; | ||
samm:preferredName "Population Filter"@en ; | ||
samm:description "Filters on the common early warning data set to specify a population"@en ; | ||
samm:characteristic :PopulationFilterCharateristic . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo Charateristic
|
||
:uniqueIdentifiers a samm:Property ; | ||
samm:preferredName "listOfSerialNumbers"@en ; | ||
samm:description "List of unique identifiers to explicitly specifiy parts or vehicles exhibiting the issue described in the property \"anomalyDescription\""@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo specifiy
samm:characteristic :UniqueIdentifiersCharacteristic . | ||
|
||
:PopulationFilterCharateristic a samm-c:List ; | ||
samm:preferredName "Population Filter Charateristic"@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo Charateristic
|
||
:ValuesCharacteristic a samm-c:List ; | ||
samm:preferredName "Values Characteristic"@en ; | ||
samm:description "Caracteristic for the list of values"@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo Caracteristic
|
||
:qualityAppFilter a samm:Property ; | ||
samm:preferredName "Quality App Filter"@en ; | ||
samm:description "A filter to define the population of affected parts in a C-atnea-X quality app"@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is C-atnea-X
right?
samm:characteristic :PopulationFilterCharateristic . | ||
|
||
:uniqueIdentifiers a samm:Property ; | ||
samm:preferredName "listOfSerialNumbers"@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
preferredName
should be plain english and not camel case.
:catenaXId a samm:Property ; | ||
samm:preferredName "Catena-X ID"@en ; | ||
samm:description "A unique identifier for a part or a vehicle. The identifier relates to a Catena-X digital twin"@en ; | ||
samm:characteristic samm-c:Text ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can make use of shared aspect model (io.catenax.shared.uuid:1.0.0) for UUID.
you can refer this https://github.com/eclipse-tractusx/sldt-semantic-models/blob/a07bdd5b1515767d4f8d07deb47cf2a4aa728bef/io.catenax.quality_message_content/1.0.0/QualityMessageContent.ttl#L35C9-L35C17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Timothy-Ra please check this also.
io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttl
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check my comments
accoridng to aspect model stuart
changed urn from example to specific io.catenax.early_warning_notification
accoring to feedback from aspect model stuart
model simplified
@agg3fe we changed the model according to your remarks and made some changes due to new infos on our side. Please check once again. Thank you :) |
@@ -0,0 +1,158 @@ | |||
####################################################################### | |||
# Copyright (c) 2022 Robert Bosch GmbH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please change the year to 2024
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> . | ||
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> . | ||
@prefix : <urn:samm:io.catenax.early_warning_notification:1.0.0#> . | ||
@prefix ext-header: <urn:samm:io.catenax.shared.message_header:1.0.0#> . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can consider reference to new message header 2.0.0 as new version is released now. You might need to update your branch with main branch.
@Timothy-Ra please check my comments |
Validation Report for io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttlInput model is valid |
update accoring to updated notification header
@andreas-hutter and I updated according to your comments and some last minute changes from our side. @agg3fe please validade |
samm:characteristic samm-c:Text ; | ||
samm:exampleValue "3747429FGH382923974682" . | ||
|
||
:catenaXId a samm:Property ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this is a unique catenaXId, you can make use of shared aspect model Uuid.
example: https://github.com/eclipse-tractusx/sldt-semantic-models/blob/main/io.catenax.asset_tracker_links/2.0.0/AssetTrackerLinks.ttl#L38
please update your PR with main branch and use Uuid2.0.0 version instead of 1.0.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, it´s not a catenaXId.
The VAN is the anomynized VIN (vehicle identification number). Which is the identifier for each individual car. You find it in the botton right corner of the wind shield of each car. Each OEM may uses another hash algorithm so that we musn´t define any further semantic limitations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay, i misunderstood, I changed it
Validation Report for io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttlInput model is valid |
updated Catena-X ID reference
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Validation Report for io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttlSyntax error in line 125, column 24: Undefined prefix: ext-uuid 120: samm:exampleValue "3747429FGH382923974682" .
121:
122: :catenaXId a samm:Property ;
123: samm:preferredName "Catena-X ID"@en ;
124: samm:description "A unique identifier for a part or a vehicle. The identifier relates to a Catena-X digital twin"@en ;
->125: samm:characteristic ext-uuid:UuidV4Trait ;
126: samm:exampleValue "urn:uuid:ed85f17e-29dd-473c-9cb8-d7ad1dc44d2f" .
127:
128: :serialPartNumber a samm:Property ;
129: samm:preferredName "Serial Part Number"@en ;
130: samm:description "A unique serial number for a produced part"@en ; |
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> . | ||
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> . | ||
@prefix : <urn:samm:io.catenax.early_warning_notification:1.0.0#> . | ||
@prefix ext-header: <urn:samm:io.catenax.shared.message_header:2.0.0#> . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you did not add the dependency for uuid here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add dependency
Hi @Timothy-Ra, As I suggested above. please add the dependency which is missing. your model is invalid right now. |
Added the requested reference to <urn:samm:io.catenax.shared.uuid:2.0.0> Deleted 3 properties that were actually not needed.
I have added the dependecy as requested (and removed 3 no longer needed properties). Can you please check the model validity again ? |
Validation Report for io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttlInput model is valid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@Timothy-Ra please check eclipsefdn/eca step https://api.eclipse.org/git/eca/status/gh/eclipse-tractusx/sldt-semantic-models/484. |
All notable changes to this model will be documented in this file. | ||
|
||
[Unreleased] | ||
[1.0.0] - 2023-12-11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please change it to today's date if you need MS3 today.
Updated date
Validation Report for io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttlInput model is valid |
Converted notificationID characteristic to ext-uuid. Deleted unused dependency on share notification header.
Sorry, I had a copy paste error before.
Validation Report for io.catenax.early_warning_notification/1.0.0/EarlyWarningNotification.ttlInput model is valid |
thanks so much @agg3fe and @andreas-hutter for the great cooperation and that we finished that model :) |
Description
The new model enables the notification function about findings from joint field monitoring between two participants of the supply chain. This functionality is part of the early warning feature from the use case quality.
Closes #421
MS2 Criteria
(to be filled out by PR reviewer)
DismantlerId
andDismantlerName
use an EntityDismantler
with the propertiesname
andid
or use a URN likeio.catenax.dismantler:0.0.1
)preferredName
anddescription
are not the samepreferredName
should be human readable and follow normal orthography (e.g., no camel case but normal word separation)MS3 Criteria
(to be filled out by semantic modeling team before merge to main-branch)