-
Notifications
You must be signed in to change notification settings - Fork 12
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
SAMM-CLI aas generation: Mapping of Set Characteristics #597
base: main
Are you sure you want to change the base?
SAMM-CLI aas generation: Mapping of Set Characteristics #597
Conversation
...odel-aas-generator/src/main/java/org/eclipse/esmf/aspectmodel/aas/AspectModelAasVisitor.java
Outdated
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.
see finding from Andreas: it seems the characteristic is not taken into account when determining the typeValueElementList value.
...odel-aas-generator/src/main/java/org/eclipse/esmf/aspectmodel/aas/AspectModelAasVisitor.java
Show resolved
Hide resolved
* @return the transformed string in the format "_role[8_characters_of_hash]" | ||
* @throws NoSuchAlgorithmException if the SHA-256 algorithm is not available | ||
*/ | ||
private String transformEnumerationValue( final String enumValue ) throws NoSuchAlgorithmException { |
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.
How would the mapping work in the opposite direction with this approach?
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 think, that I have to add separate method in the opposite direction.
Thanks!
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 did not check the complete code, but see some comments/questions
defaultProperty.setIdShort( context.getPropertyShortId() ); | ||
defaultProperty.setValueType( AasDataTypeMapper.mapAspectTypeToAasXsdDataType( mapType( type ) ) ); | ||
defaultProperty.setDisplayName( LangStringMapper.NAME.map( property.getPreferredNames() ) ); | ||
defaultProperty.setSemanticId( buildPropertyReferenceToGlobalReference( 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.
Why propertyReference?
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.
Could you please explain more, because in this method "buildPropertyReferenceToGlobalReference" we building GlobalReference, not property reference?
Description
Changes:
Fixes #481
Type of change
Checklist: