-
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]: Shared PartSiteInformationAsBuilt 1.0.0 #453
[New Model]: Shared PartSiteInformationAsBuilt 1.0.0 #453
Conversation
Validation Report for io.catenax.shared.part_site_information/1.0.0/PartSiteInformation.ttlInput model is valid |
@johannsvarela do you want to add 'PartSiteInformationAsPlanned.ttl' also? If yes, then you have to create a separate Pull Request for this. we can have only one model (.ttl) file in one PR for new model. |
@agg3fe The second ttl-File (PartSiteInformationAsPlanned) contain the migration from BAMM to SAMM for the model. Since the PartSiteInformationAsPlanned will be set to 'deprecate', i don't know if its necessary to migrate to SAMM? Otherwise i will integrate the SAMM-migration to the existing deprecation PR #454. Would it be fine for you? |
If it is deprecated, then you don't need to migrate it from BAMM to SAMM. The separate PR is fine for me to set it to deprecate. you can remove the file from PR. |
Validation Report for io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttlInput model is valid |
io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttl
Outdated
Show resolved
Hide resolved
io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttl
Show resolved
Hide resolved
io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.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
Validation Report for io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttlInput model is valid |
@agg3fe The changes were implemented as you suggested! :-) |
Validation Report for io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttlInput model is valid |
|
||
All notable changes to this model will be documented in this file. | ||
|
||
## [1.0.0] 2023-11-27 |
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 change the date here to 20th if you want MS3 approval today.
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.
I will change the date after we have introduced and discussed this model in our semantic group.
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
modeling_team |
@agg3fe Jira-Issue to generate the testdata set for this model was created: https://jira.catena-x.net/browse/TDG-35 |
Based on the results / definition of the Industry Core, the part site information is also needed in the DTasBuilt context. Therefore a new aspect model was created, which contain the information of the sites of a part. The new model will be implemented as shared aspect model who will be integrated into the existing models SerialPart, Batch and JiS in further steps.
Closes #442
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)