eclipse-tractusx / sldt-semantic-models

sldt-semantic-models
Creative Commons Attribution 4.0 International
24 stars 41 forks source link

[New Model]: Shared PartSiteInformationAsBuilt 1.0.0 #453

Closed johannsvarela closed 6 months ago

johannsvarela commented 7 months ago

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)

MS3 Criteria

(to be filled out by semantic modeling team before merge to main-branch)

github-actions[bot] commented 7 months ago

Validation Report for io.catenax.shared.part_site_information/1.0.0/PartSiteInformation.ttl

Input model is valid

johannsvarela commented 7 months ago

FYI @jonbckr @BenediktMuellerIML @jacewski-bosch

agg3fe commented 7 months ago

@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.

johannsvarela commented 7 months ago

@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?

agg3fe commented 7 months ago

@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.

github-actions[bot] commented 7 months ago

Validation Report for io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttl

Input model is valid

github-actions[bot] commented 6 months ago

Validation Report for io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttl

Input model is valid

johannsvarela commented 6 months ago

Please check my comments

@agg3fe The changes were implemented as you suggested! :-)

github-actions[bot] commented 6 months ago

Validation Report for io.catenax.shared.part_site_information_as_built/1.0.0/PartSiteInformationAsBuilt.ttl

Input model is valid

johannsvarela commented 6 months ago

modeling_team

johannsvarela commented 6 months ago

@agg3fe Jira-Issue to generate the testdata set for this model was created: https://jira.catena-x.net/browse/TDG-35