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

[openweathermap] Semantics added to channel types #10972

Merged
merged 1 commit into from
Feb 8, 2022

Conversation

lolodomo
Copy link
Contributor

Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier lg.hc@free.fr

@lolodomo lolodomo requested a review from cweitkamp as a code owner July 11, 2021 10:08
@lolodomo lolodomo force-pushed the owm_semantics branch 2 times, most recently from bf8a8f6 to e0581fb Compare July 11, 2021 12:21
@Skinah Skinah added the enhancement An enhancement or new feature for an existing add-on label Jul 12, 2021
@@ -399,6 +440,10 @@
<label>Forecasted Temperature</label>
<description>Forecasted outdoor temperature.</description>
<category>Temperature</category>
<tags>
<tag>Measurement</tag>
Copy link
Member

Choose a reason for hiding this comment

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

I am wondering here: A forecast is not really a measurement. I'd therefore rather refrain from adding this tag.
We might want to think of something like a "forecast" tag for the tag library instead. Wdyt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, a forecast is not exactly a measurement.
I will add a new Point "Forecast" in the semantics tag.

Copy link
Contributor Author

@lolodomo lolodomo Jul 12, 2021

Choose a reason for hiding this comment

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

I am going to replace "Measurement" by "Status" for forecast channels as semantics library is frozen.

@kaikreuzer
Copy link
Member

I'd suggest we park this PR until we finish the discussion on openhab/openhab-core#1791. Putting "Status" on everything where we don't have dedicated semantic tags for does not feel right to me and I wouldn't want to see this to be done for 300+ bindings now...

@lolodomo
Copy link
Contributor Author

lolodomo commented Jul 20, 2021

@kaikreuzer : to make progress, I propose to only keep tags that are obvious (in particular measurements like temperature, humidity, pressure, ...) and suppress "Status" where we don't have dedicated semantic tags. Can we at least do that ?

@cweitkamp cweitkamp added the awaiting feedback Awaiting feedback from the pull request author label Oct 16, 2021
@lolodomo
Copy link
Contributor Author

@cweitkamp : the "awaiting feedback" is for @kaikreuzer , right ?

@cweitkamp
Copy link
Contributor

Kind of. It is more or less feedback from discussion in openhab/openhab-core#1791.

Didn't find a proper label for it.

@kaikreuzer
Copy link
Member

Sorry for dropping the ball here. 😩

to make progress, I propose to only keep tags that are obvious (in particular measurements like temperature, humidity, pressure, ...) and suppress "Status" where we don't have dedicated semantic tags. Can we at least do that ?

Yes, let's do that, please!

@lolodomo lolodomo removed the awaiting feedback Awaiting feedback from the pull request author label Feb 6, 2022
Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo
Copy link
Contributor Author

lolodomo commented Feb 8, 2022

@kaikreuzer : PR is updated.

@kaikreuzer kaikreuzer merged commit 99cfb65 into openhab:main Feb 8, 2022
@kaikreuzer kaikreuzer added this to the 3.3 milestone Feb 8, 2022
@lolodomo lolodomo deleted the owm_semantics branch February 8, 2022 18:07
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Apr 27, 2022
Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jun 29, 2022
Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Feb 23, 2023
Also fix few categories to use only official categories

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
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 for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants