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

[semantics] Added properties Distance and Angle #2021

Conversation

Rosi2143
Copy link
Contributor

Proposal is to add Distance and Angle as additional property
this allows to use these basic units for items.

Signed-off-by: Michael Roßner [email protected]

Proposal is to add Distance and Angle as additional property
this allows to use these basic units for items.

Signed-off-by: Michael Roßner <[email protected]>
@Rosi2143 Rosi2143 requested a review from a team as a code owner December 28, 2020 16:41
@cweitkamp cweitkamp added rebuild Triggers the Jenkins PR build and removed rebuild Triggers the Jenkins PR build labels Dec 29, 2020
@cweitkamp cweitkamp added the enhancement An enhancement or new feature of the Core label Jan 16, 2021
Base automatically changed from master to main January 18, 2021 20:04
@cweitkamp cweitkamp changed the title Semantic: Add properties Distance and Angle [semantics] Added properties Distance and Angle Mar 7, 2021
@cweitkamp
Copy link
Contributor

Thank you very much for your first contribution. And sorry for our late response.

May I ask you to prepare a related PR in https://github.com/openhab/openhab-webui/ (similar to openhab/openhab-webui#537)?

@@ -0,0 +1,27 @@
/**
* Copyright (c) 2010-2020 Contributors to the openHAB project
Copy link
Member

Choose a reason for hiding this comment

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

This looks like an outdated license header (saying 2020 instead of 2021). Does our generator script need an update?

@kaikreuzer
Copy link
Member

@Rosi2143 & @cweitkamp: I didn't follow the process of the ontology expansion in detail, so I wonder if this PR is aligned with #1791, which seems to be the "coordinating issue" for such changes?

@cweitkamp
Copy link
Contributor

I didn't follow #1791 in detail either. Thus I do not know if the list in the OP is fix or final or just an intermediate wrap up. Both properties are not mentioned in the discussion but from my pov the properties (for Point -> Measurements) should be aligned - beside others - with the list of dimensions provided by OHC UoM model.

@cweitkamp cweitkamp added the stale As soon as a PR is marked stale, it can be removed 6 months later. label Oct 16, 2021
@Rosi2143
Copy link
Contributor Author

I follow up on it with proposed #1791

@Rosi2143 Rosi2143 closed this Dec 12, 2021
@Rosi2143 Rosi2143 mentioned this pull request Dec 12, 2021
65 tasks
@Rosi2143 Rosi2143 deleted the Rosi2143/Extend_Semantic_with_Distance_and_Angle branch December 14, 2021 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core semantics stale As soon as a PR is marked stale, it can be removed 6 months later.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants