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

Introduce OID attributes #340

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Introduce OID attributes #340

merged 1 commit into from
Sep 10, 2024

Conversation

andlaus
Copy link
Collaborator

@andlaus andlaus commented Sep 10, 2024

OID stands for "Object ID" and is meant to be a UUID for objects, i.e. it is supposed to be an invariant globally unique identifier for objects. It can e.g., be used to locate objects in an ODX editor across projects, but there is no formal mechanism to reference an object via its OID within the ODX data model.

Andreas Lauser <andreas.lauser@mercedes-benz.com>, on behalf of MBition GmbH.
Provider Information

OID stands for "Object ID" and is meant to be a UUID for objects,
i.e. it is supposed to be an invariant globally unique identifier for
objects. It can e.g., be used to locate objects in an ODX editor
across projects, but there is no formal mechanism to reference an
object via its OID within the ODX data model.

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Gerrit Ecke <gerrit.ecke@mbition.io>
@@ -40,6 +40,7 @@ class Parameter(NamedElement):
define any non-positionable parameter types.

"""
oid: Optional[str]
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note that, for whatever reason, most parameters extending POSITIONABLE-PARAM do not exhibit an ID but an OID. AFAICS, the only parameters exhibiting IDs are LENGTH-KEY and TABLE-KEY. (why oh why?)

@andlaus andlaus merged commit b452b60 into mercedes-benz:main Sep 10, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants