-
Notifications
You must be signed in to change notification settings - Fork 27
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: Rename GlobalReference
to ExternalReference
#106
model.base: Rename GlobalReference
to ExternalReference
#106
Conversation
GlobalReference
to ExternalReference
GlobalReference
to ExternalReference
README.md
Outdated
id_short='ExampleProperty', # Identifying string of the element within the submodel namespace | ||
value_type=model.datatypes.String, # Data type of the value | ||
value='exampleValue', # Value of the property | ||
semantic_id=semantic_reference # set the semantic reference |
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 have these lines been re-indented with 2 spaces instead of 4?
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 honestly no idea. Must have been PyCharm during rebasing. I reverted the indentation.
This should be rebased after #107 is merged and changed such that it replaces the new occurances of |
I've updated this branch to the newest changes in
still need to be adapted manually. @jkhsjdhjs could you do this? |
basyx/aas/model/base.py
Outdated
@@ -777,7 +777,7 @@ class Reference(metaclass=abc.ABCMeta): | |||
A reference is an ordered list of keys, each key referencing an element. The complete list of keys may for | |||
example be concatenated to a path that then gives unique access to an element or entity. | |||
|
|||
This is the abstract superclass of GlobalReference and ModelReference, which implements common attributes and | |||
This is the abstract superclass of External and ModelReference, which implements common attributes and |
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.
This External
should be changed to ExternalReference
, as it references the class.
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.
Oh, I introduced this change 😂
0bc0134
to
1081a70
Compare
Version 3.0 of the spec renames `ReferenceTypes/GlobalReference` to `ReferenceTypes/ExternalReference` to avoid confusion with `KeyTypes/GlobalReference`.
… to `ExternalReference`
1081a70
to
5be0392
Compare
GlobalReference
to ExternalReference
GlobalReference
to ExternalReference
…ence Version 3.0 of the spec renames ReferenceTypes/GlobalReference to ReferenceTypes/ExternalReference to avoid confusion with KeyTypes/GlobalReference. This merge implements these changes.
Version 3.0 of the spec renames
ReferenceTypes/GlobalReference
toReferenceTypes/ExternalReference
to avoid confusion withKeyTypes/GlobalReference
.Todo: Adapt the examples in
test/compliance_tool/files
.