-
Notifications
You must be signed in to change notification settings - Fork 42
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
change bamm to samm #760
base: main
Are you sure you want to change the base?
change bamm to samm #760
Conversation
@gautam-e The version for this model should be major update, means it should be 3.0.0 (as 2.0.0 was already removed). |
@agg3fe changes done. Please check if ok. |
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.
Thank you for updating the BAMM version to SAMM. From my side only minor changes are requested regarding:
- all model elements at least contain the fields "preferred name" and "description" in English language. The description must be comprehensible. It is not required to write full sentences but style should be consistent over the whole model
- preferredName should be human readable and follow normal orthography (e.g., no camel case but normal word separation)
- all properties with an simple type have an example value
|
||
:IdentifierCharacteristic a samm:Characteristic; | ||
samm:preferredName "IdentifierCharacteristic"@en; | ||
samm:dataType xsd:string . |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
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.
@jSchuetz88 do you know where I can get this from?
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 have made changes except for the example part
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.
@jSchuetz88 I need help here. Don't know what to do.
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.
Thank you for addressing the comments. And sorry for the confusion, this characteristic does not need to provide an example value.
This comment is resolved from my side.
:partDescription a samm:Property; | ||
samm:preferredName "part description"@en; | ||
samm:description "The description of the part in the primary language of the production facility of the product owner."@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.
No description provided for simple type.
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.
@jSchuetz88 I need help here. Don't know what to do.
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.
Sorry, i meant "no example value provided". Unfortunately I lack the business knowledge in that case. Based on other semantic models, it could be something like:
the example from the PCF data model:
samm:exampleValue "Ethanol, 95% solution" .
or an other one from: SingleLevelBomAsSpecified
samm:exampleValue "The steering wheel is nice and round".
@jSchuetz88 I have made the changes that I can make. I would need some help for the remaining suggestions. |
@gautam-e Could you please check eclipsefdn/eca step in the work flow. Looks like you have not signed it. |
@gautam-e Could you please join the Weekly Monday meeting. We can discuss the Pull Request and provide approval. Let me know if you have meeting invite or not. |
Hi @gautam-e, sorry for the late reply. If you provide me your E-Mail-Adress, I will invide you to the meeting? |
Description
This addresses #744.
bamm
has been renamed tosamm
and the version has been bumped (patch).-->
Closes #744
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)