Skip to content
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

model.base: improve NamespaceSet #154

Closed
wants to merge 28 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
77e0f86
Fix Entity.specificAssetIds in schemas
zrgt Oct 27, 2023
e1fffec
- Set `Entity.specific_asset_id` as `Iterable[SpecificAssetId]`, beca…
zrgt Oct 27, 2023
92e8618
Refactor `check_entity_equal()`&`check_asset_information_equal()`
zrgt Oct 27, 2023
0ed5d4f
Update test files
zrgt Oct 27, 2023
bc66b2a
Update tutorial_serialization_deserialization.py
zrgt Oct 27, 2023
91bfe92
Use getter/setter decorators for global_asset_id
zrgt Oct 29, 2023
0386628
Fix errors from MyPy
zrgt Oct 29, 2023
2f8a731
Fix pycodestyle errors
zrgt Oct 29, 2023
f537658
Fix typehint of specific_asset_id
zrgt Nov 3, 2023
046ee15
Fix wrong example value
zrgt Nov 3, 2023
7a2e341
model.base: fix `ConstrainedList.clear()` atomicity
jkhsjdhjs Nov 4, 2023
ff31f45
model.submodel: remove `_string_constraints` decorator from `Entity`
jkhsjdhjs Nov 4, 2023
7ac9aa6
model.submodel: move `Entity.global_asset_id` string constraint check…
jkhsjdhjs Nov 4, 2023
4b8a7ae
model.submodel: remove duplicate code in `Entity`
jkhsjdhjs Nov 4, 2023
3dcf9ff
model.ass: add `_validate_asset_ids()` function to `AssetInformation`
jkhsjdhjs Nov 4, 2023
b853160
model.{aas, submodel}: add set hook to `{AssetInformation, Entity}.sp…
jkhsjdhjs Nov 4, 2023
39a8c48
model.submodel: assign values correctly in `Entity.__init__`
jkhsjdhjs Nov 4, 2023
6b3de29
model.{aas, submodel}: add getter/setter for `{AssetInformation, Enti…
jkhsjdhjs Nov 4, 2023
47b2e47
model.submodel: verify constraints when `SpecificAssetIds` are added …
jkhsjdhjs Nov 4, 2023
5e0f160
model.{aas, submodel}: improve `{AssetInformation, Entity}.global_ass…
jkhsjdhjs Nov 4, 2023
6fb375d
test.model: add `AssetInformation` tests and improve `Entity` tests
jkhsjdhjs Nov 4, 2023
f11c7f5
Merge branch 'fix/constrained_list_clear_atomicity' into entityfix
jkhsjdhjs Nov 4, 2023
f8a2bf6
Merge pull request #1 from rwth-iat/entityfix
zrgt Nov 7, 2023
9ed5704
Refactor `Entity` and `AssetInformation`
zrgt Nov 7, 2023
0222237
Refactor `Entity` and `AssetInformation`
zrgt Nov 7, 2023
130eb89
Merge branch 'improve/V30' into update/documentation
zrgt Nov 8, 2023
a0184e5
Refactor NamespaceSet
zrgt Nov 8, 2023
ddc425b
Fix of namespace and its tests
zrgt Nov 8, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions basyx/aas/adapter/json/aasJSONSchema.json
Original file line number Diff line number Diff line change
Expand Up @@ -444,8 +444,12 @@
"maxLength": 2000,
"pattern": "^([\\t\\n\\r -\ud7ff\ue000-\ufffd]|\\ud800[\\udc00-\\udfff]|[\\ud801-\\udbfe][\\udc00-\\udfff]|\\udbff[\\udc00-\\udfff])*$"
},
"specificAssetId": {
"$ref": "#/definitions/SpecificAssetId"
"specificAssetIds": {
"type": "array",
"items": {
"$ref": "#/definitions/SpecificAssetId"
},
"minItems": 1
}
},
"required": [
Expand Down
19 changes: 13 additions & 6 deletions basyx/aas/adapter/json/json_deserialization.py
Original file line number Diff line number Diff line change
Expand Up @@ -417,13 +417,19 @@ def _construct_value_reference_pair(cls, dct: Dict[str, object],
@classmethod
def _construct_asset_information(cls, dct: Dict[str, object], object_class=model.AssetInformation)\
-> model.AssetInformation:
ret = object_class(asset_kind=ASSET_KIND_INVERSE[_get_ts(dct, 'assetKind', str)])
cls._amend_abstract_attributes(ret, dct)
global_asset_id = None
if 'globalAssetId' in dct:
ret.global_asset_id = _get_ts(dct, 'globalAssetId', str)
global_asset_id = _get_ts(dct, 'globalAssetId', str)
specific_asset_id = set()
if 'specificAssetIds' in dct:
for desc_data in _get_ts(dct, "specificAssetIds", list):
ret.specific_asset_id.add(cls._construct_specific_asset_id(desc_data, model.SpecificAssetId))
specific_asset_id.add(cls._construct_specific_asset_id(desc_data, model.SpecificAssetId))

ret = object_class(asset_kind=ASSET_KIND_INVERSE[_get_ts(dct, 'assetKind', str)],
global_asset_id=global_asset_id,
specific_asset_id=specific_asset_id)
cls._amend_abstract_attributes(ret, dct)

if 'assetType' in dct:
ret.asset_type = _get_ts(dct, 'assetType', str)
if 'defaultThumbnail' in dct:
Expand Down Expand Up @@ -497,9 +503,10 @@ def _construct_entity(cls, dct: Dict[str, object], object_class=model.Entity) ->
global_asset_id = None
if 'globalAssetId' in dct:
global_asset_id = _get_ts(dct, 'globalAssetId', str)
specific_asset_id = None
specific_asset_id = set()
if 'specificAssetIds' in dct:
specific_asset_id = cls._construct_specific_asset_id(_get_ts(dct, 'specificAssetIds', dict))
for desc_data in _get_ts(dct, "specificAssetIds", list):
specific_asset_id.add(cls._construct_specific_asset_id(desc_data, model.SpecificAssetId))

ret = object_class(id_short=None,
entity_type=ENTITY_TYPES_INVERSE[_get_ts(dct, "entityType", str)],
Expand Down
2 changes: 1 addition & 1 deletion basyx/aas/adapter/json/json_serialization.py
Original file line number Diff line number Diff line change
Expand Up @@ -631,7 +631,7 @@ def _entity_to_json(cls, obj: model.Entity) -> Dict[str, object]:
if obj.global_asset_id:
data['globalAssetId'] = obj.global_asset_id
if obj.specific_asset_id:
data['specificAssetIds'] = obj.specific_asset_id
data['specificAssetIds'] = list(obj.specific_asset_id)
return data

@classmethod
Expand Down
8 changes: 7 additions & 1 deletion basyx/aas/adapter/xml/AAS.xsd
Original file line number Diff line number Diff line change
Expand Up @@ -284,7 +284,13 @@
</xs:restriction>
</xs:simpleType>
</xs:element>
<xs:element name="specificAssetId" type="specificAssetId_t" minOccurs="0" maxOccurs="1"/>
<xs:element name="specificAssetIds" minOccurs="0" maxOccurs="1">
<xs:complexType>
<xs:sequence>
<xs:element name="specificAssetId" type="specificAssetId_t" minOccurs="1" maxOccurs="unbounded"/>
</xs:sequence>
</xs:complexType>
</xs:element>
</xs:sequence>
</xs:group>
<xs:group name="environment">
Expand Down
28 changes: 17 additions & 11 deletions basyx/aas/adapter/xml/xml_deserialization.py
Original file line number Diff line number Diff line change
Expand Up @@ -797,13 +797,17 @@ def construct_capability(cls, element: etree.Element, object_class=model.Capabil

@classmethod
def construct_entity(cls, element: etree.Element, object_class=model.Entity, **_kwargs: Any) -> model.Entity:
global_asset_id = _get_text_or_none(element.find(NS_AAS + "globalAssetId"))
specific_asset_id = _failsafe_construct(element.find(NS_AAS + "specificAssetId"),
cls.construct_specific_asset_id, cls.failsafe)
specific_asset_id = set()
specific_assset_ids = element.find(NS_AAS + "specificAssetIds")
if specific_assset_ids is not None:
for id in _child_construct_multiple(specific_assset_ids, NS_AAS + "specificAssetId",
cls.construct_specific_asset_id, cls.failsafe):
specific_asset_id.add(id)

entity = object_class(
id_short=None,
entity_type=_child_text_mandatory_mapped(element, NS_AAS + "entityType", ENTITY_TYPES_INVERSE),
global_asset_id=global_asset_id,
global_asset_id=_get_text_or_none(element.find(NS_AAS + "globalAssetId")),
specific_asset_id=specific_asset_id)

if not cls.stripped:
Expand Down Expand Up @@ -994,17 +998,19 @@ def construct_specific_asset_id(cls, element: etree.Element, object_class=model.
@classmethod
def construct_asset_information(cls, element: etree.Element, object_class=model.AssetInformation, **_kwargs: Any) \
-> model.AssetInformation:
asset_information = object_class(
_child_text_mandatory_mapped(element, NS_AAS + "assetKind", ASSET_KIND_INVERSE),
)
global_asset_id = _get_text_or_none(element.find(NS_AAS + "globalAssetId"))
if global_asset_id is not None:
asset_information.global_asset_id = global_asset_id
specific_asset_id = set()
specific_assset_ids = element.find(NS_AAS + "specificAssetIds")
if specific_assset_ids is not None:
for id in _child_construct_multiple(specific_assset_ids, NS_AAS + "specificAssetId",
cls.construct_specific_asset_id, cls.failsafe):
asset_information.specific_asset_id.add(id)
specific_asset_id.add(id)

asset_information = object_class(
_child_text_mandatory_mapped(element, NS_AAS + "assetKind", ASSET_KIND_INVERSE),
global_asset_id=_get_text_or_none(element.find(NS_AAS + "globalAssetId")),
specific_asset_id=specific_asset_id,
)

asset_type = _get_text_or_none(element.find(NS_AAS + "assetType"))
if asset_type is not None:
asset_information.asset_type = asset_type
Expand Down
5 changes: 4 additions & 1 deletion basyx/aas/adapter/xml/xml_serialization.py
Original file line number Diff line number Diff line change
Expand Up @@ -801,7 +801,10 @@ def entity_to_xml(obj: model.Entity,
if obj.global_asset_id:
et_entity.append(_generate_element(NS_AAS + "globalAssetId", text=obj.global_asset_id))
if obj.specific_asset_id:
et_entity.append(specific_asset_id_to_xml(obj.specific_asset_id, NS_AAS + "specificAssetId"))
et_specific_asset_id = _generate_element(name=NS_AAS + "specificAssetIds")
for specific_asset_id in obj.specific_asset_id:
et_specific_asset_id.append(specific_asset_id_to_xml(specific_asset_id, NS_AAS + "specificAssetId"))
et_entity.append(et_specific_asset_id)
return et_entity


Expand Down
42 changes: 17 additions & 25 deletions basyx/aas/examples/data/_helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -454,7 +454,7 @@ def _find_reference(self, object_: model.Reference, search_list: Iterable) -> Un
return element
return None

def _find_specific_asset_id(self, object_: model.SpecificAssetId, search_list: Union[Set, List]) \
def _find_specific_asset_id(self, object_: model.SpecificAssetId, search_list: Iterable) \
-> Union[model.SpecificAssetId, None]:
"""
Find a SpecificAssetId in an list
Expand Down Expand Up @@ -604,25 +604,26 @@ def check_entity_equal(self, object_: model.Entity, expected_value: model.Entity
self._check_abstract_attributes_submodel_element_equal(object_, expected_value)
self.check_attribute_equal(object_, 'entity_type', expected_value.entity_type)
self.check_attribute_equal(object_, 'global_asset_id', expected_value.global_asset_id)
if object_.specific_asset_id and expected_value.specific_asset_id:
self.check_specific_asset_id(object_.specific_asset_id, expected_value.specific_asset_id)
else:
if expected_value.specific_asset_id:
self.check(expected_value.specific_asset_id is not None,
'SpecificAssetId {} must exist'.format(repr(expected_value.specific_asset_id)),
value=object_.specific_asset_id)
else:
if object_.specific_asset_id:
self.check(expected_value.specific_asset_id is None, 'Enity {} must not have a '
'specificAssetId'.format(repr(object_)),
value=expected_value.specific_asset_id)
self._check_specific_asset_ids_equal(object_.specific_asset_id, expected_value.specific_asset_id, object_)
self.check_contained_element_length(object_, 'statement', model.SubmodelElement, len(expected_value.statement))
for expected_element in expected_value.statement:
element = object_.get_referable(expected_element.id_short)
self.check(element is not None, 'Entity {} must exist'.format(repr(expected_element)))
self.check(element is not None, f'Entity {repr(expected_element)} must exist')

found_elements = self._find_extra_elements_by_id_short(object_.statement, expected_value.statement)
self.check(found_elements == set(), 'Enity {} must not have extra statements'.format(repr(object_)),
self.check(found_elements == set(), f'Enity {repr(object_)} must not have extra statements',
value=found_elements)

def _check_specific_asset_ids_equal(self, object_: Iterable[model.SpecificAssetId],
expected_value: Iterable[model.SpecificAssetId],
object_parent):
for expected_pair in expected_value:
pair = self._find_specific_asset_id(expected_pair, object_)
if self.check(pair is not None, f'SpecificAssetId {repr(expected_pair)} must exist'):
self.check_specific_asset_id(pair, expected_pair) # type: ignore

found_elements = self._find_extra_object(object_, expected_value, model.SpecificAssetId)
self.check(found_elements == set(), f'{repr(object_parent)} must not have extra specificAssetIds',
value=found_elements)

def _check_event_element_equal(self, object_: model.EventElement, expected_value: model.EventElement):
Expand Down Expand Up @@ -722,16 +723,7 @@ def check_asset_information_equal(self, object_: model.AssetInformation, expecte
self.check_attribute_equal(object_, 'global_asset_id', expected_value.global_asset_id)
self.check_contained_element_length(object_, 'specific_asset_id', model.SpecificAssetId,
len(expected_value.specific_asset_id))
for expected_pair in expected_value.specific_asset_id:
pair = self._find_specific_asset_id(expected_pair, object_.specific_asset_id)
if self.check(pair is not None, 'SpecificAssetId {} must exist'.format(repr(expected_pair))):
self.check_specific_asset_id(pair, expected_pair) # type: ignore

found_elements = self._find_extra_object(object_.specific_asset_id, expected_value.specific_asset_id,
model.SpecificAssetId)
self.check(found_elements == set(), '{} must not have extra '
'specificAssetIds'.format(repr(object_)),
value=found_elements)
self._check_specific_asset_ids_equal(object_.specific_asset_id, expected_value.specific_asset_id, object_)
self.check_attribute_equal(object_, 'asset_type', object_.asset_type)
if object_.default_thumbnail and expected_value.default_thumbnail:
self.check_resource_equal(object_.default_thumbnail, expected_value.default_thumbnail)
Expand Down
13 changes: 7 additions & 6 deletions basyx/aas/examples/data/example_aas.py
Original file line number Diff line number Diff line change
Expand Up @@ -244,11 +244,12 @@ def create_example_bill_of_material_submodel() -> model.Submodel:
entity_type=model.EntityType.SELF_MANAGED_ENTITY,
statement={submodel_element_property, submodel_element_property2},
global_asset_id='http://acplt.org/TestAsset/',
specific_asset_id=model.SpecificAssetId(name="TestKey",
value="TestValue",
external_subject_id=model.ExternalReference(
(model.Key(type_=model.KeyTypes.GLOBAL_REFERENCE,
value='http://acplt.org/SpecificAssetId/'),))),
specific_asset_id={
model.SpecificAssetId(name="TestKey", value="TestValue",
external_subject_id=model.ExternalReference(
(model.Key(type_=model.KeyTypes.GLOBAL_REFERENCE,
value='http://acplt.org/SpecificAssetId/'),))
)},
category="PARAMETER",
description=model.MultiLanguageTextType({
'en-US': 'Legally valid designation of the natural or judicial person which '
Expand Down Expand Up @@ -276,7 +277,7 @@ def create_example_bill_of_material_submodel() -> model.Submodel:
entity_type=model.EntityType.CO_MANAGED_ENTITY,
statement=(),
global_asset_id=None,
specific_asset_id=None,
specific_asset_id=(),
category="PARAMETER",
description=model.MultiLanguageTextType({
'en-US': 'Legally valid designation of the natural or judicial person which '
Expand Down
3 changes: 2 additions & 1 deletion basyx/aas/examples/data/example_aas_mandatory_attributes.py
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,8 @@ def create_example_empty_asset_administration_shell() -> model.AssetAdministrati
:return: example asset administration shell
"""
asset_administration_shell = model.AssetAdministrationShell(
asset_information=model.AssetInformation(),
asset_information=model.AssetInformation(
global_asset_id='http://acplt.org/TestAsset2_Mandatory/'),
id_='https://acplt.org/Test_AssetAdministrationShell2_Mandatory')
return asset_administration_shell

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
)
aashell = model.AssetAdministrationShell(
id_='https://acplt.org/Simple_AAS',
asset_information=model.AssetInformation(),
asset_information=model.AssetInformation(global_asset_id="test"),
submodel={model.ModelReference.from_referable(submodel)}
)

Expand Down
61 changes: 49 additions & 12 deletions basyx/aas/model/aas.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ class AssetInformation:
identifiers. However, to support the corner case of very first phase of lifecycle where a stabilised/constant
global asset identifier does not already exist, the corresponding attribute “globalAssetId” is optional.

*Constraint AASd-131*: The globalAssetId or at least one specificAssetId shall be defined for AssetInformation.

:ivar asset_kind: Denotes whether the Asset is of :class:`~aas.model.base.AssetKind` "TYPE" or "INSTANCE".
Default is "INSTANCE".
:ivar global_asset_id: :class:`~aas.model.base.Identifier` modelling the identifier of the asset the AAS is
Expand All @@ -52,30 +54,65 @@ class AssetInformation:
def __init__(self,
asset_kind: base.AssetKind = base.AssetKind.INSTANCE,
global_asset_id: Optional[base.Identifier] = None,
specific_asset_id: Optional[Set[base.SpecificAssetId]] = None,
specific_asset_id: Iterable[base.SpecificAssetId] = (),
asset_type: Optional[base.Identifier] = None,
default_thumbnail: Optional[base.Resource] = None):

super().__init__()
self.asset_kind: base.AssetKind = asset_kind
self._global_asset_id: Optional[base.Identifier] = global_asset_id
self.specific_asset_id: Set[base.SpecificAssetId] = set() if specific_asset_id is None \
else specific_asset_id
self.asset_type: Optional[base.Identifier] = asset_type
self.default_thumbnail: Optional[base.Resource] = default_thumbnail
# assign private attributes, bypassing setters, as constraints will be checked below
self._specific_asset_id: base.ConstrainedList[base.SpecificAssetId] = base.ConstrainedList(
specific_asset_id,
item_set_hook=self._check_constraint_set_spec_asset_id,
item_del_hook=self._check_constraint_del_spec_asset_id
)
self._global_asset_id: Optional[base.Identifier] = global_asset_id
self._validate_global_asset_id(global_asset_id)
self._validate_aasd_131(global_asset_id, bool(specific_asset_id))

def _get_global_asset_id(self):
@property
def global_asset_id(self) -> Optional[base.Identifier]:
return self._global_asset_id

def _set_global_asset_id(self, global_asset_id: Optional[base.Identifier]):
if global_asset_id is None:
if self.specific_asset_id is None or not self.specific_asset_id:
raise ValueError("either global or specific asset id must be set")
else:
_string_constraints.check_identifier(global_asset_id)
@global_asset_id.setter
def global_asset_id(self, global_asset_id: Optional[base.Identifier]) -> None:
self._validate_global_asset_id(global_asset_id)
self._validate_aasd_131(global_asset_id, bool(self.specific_asset_id))
self._global_asset_id = global_asset_id

global_asset_id = property(_get_global_asset_id, _set_global_asset_id)
@property
def specific_asset_id(self) -> base.ConstrainedList[base.SpecificAssetId]:
return self._specific_asset_id

@specific_asset_id.setter
def specific_asset_id(self, specific_asset_id: Iterable[base.SpecificAssetId]) -> None:
# constraints are checked via _check_constraint_set_spec_asset_id() in this case
self._specific_asset_id[:] = specific_asset_id

def _check_constraint_set_spec_asset_id(self, items_to_replace: List[base.SpecificAssetId],
new_items: List[base.SpecificAssetId],
old_list: List[base.SpecificAssetId]) -> None:
self._validate_aasd_131(self.global_asset_id,
len(old_list) - len(items_to_replace) + len(new_items) > 0)

def _check_constraint_del_spec_asset_id(self, _item_to_del: base.SpecificAssetId,
old_list: List[base.SpecificAssetId]) -> None:
self._validate_aasd_131(self.global_asset_id, len(old_list) > 1)

@staticmethod
def _validate_global_asset_id(global_asset_id: Optional[base.Identifier]) -> None:
if global_asset_id is not None:
_string_constraints.check_identifier(global_asset_id)

@staticmethod
def _validate_aasd_131(global_asset_id: Optional[base.Identifier], specific_asset_id_nonempty: bool) -> None:
if global_asset_id is None and not specific_asset_id_nonempty:
raise base.AASConstraintViolation(131,
"An AssetInformation has to have a globalAssetId or a specificAssetId")
if global_asset_id is not None:
_string_constraints.check_identifier(global_asset_id)

def __repr__(self) -> str:
return "AssetInformation(assetKind={}, globalAssetId={}, specificAssetId={}, assetType={}, " \
Expand Down
Loading
Loading