openhab / openhab-core

Core framework of openHAB
https://www.openhab.org/
Eclipse Public License 2.0
915 stars 423 forks source link

Discussion: State of Thing>Location and what to do with it #2193

Open eikowagenknecht opened 3 years ago

eikowagenknecht commented 3 years ago

Current state

In OH3 we have two concepts for "Location".

  1. Semantic Model (MainUI: “Model”):
    • Location
    • Point
    • Property
    • Equipment

The Location from the Semantic Model is a new concept in OH3 and is used for various things.

  1. Things (MainUI: “Things”):
    • Location

The Location parameter of a Thing has been used for the Control area of PaperUI but is not used for anything in OH3.

The problem

The current state with two "Location" concepts is confusing (see e.g. https://community.openhab.org/t/is-location-in-a-thing-still-used-for-anything-in-oh3/116691 for some misconceptions that come up easily).

One goal of openHAB 3 is to be more attractive to novice users and those can be confused most easily.

So I think we should do something about it.

Possible ways forward

I see quite some options here:

I'm starting the discussion here because personally I would vote for a complete removal from OH or at least from the UI. That way the least confusion is caused and since it’s not used for anything, backwards compatibility shouldn’t be a major problem here. Though maybe it’s exposed through the API and some 3rd party app or addon uses it for… something? Then I’d probably go with the “show advanced” button in combination with a rename to “Physical Location” and a description that it has nothing to do with the semantic model.

But of cause that’s not my decision to make so I'd like to get the maintainers take on this.

openhab-bot commented 3 years ago

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/is-location-in-a-thing-still-used-for-anything-in-oh3/116691/16

monnimeter commented 3 years ago

In my opinion there are use cases where the "Thing Location" could be used so it makes sense; one of them I've described in #2210.

From that perspective I would vote for "Hide the Location field behind some 'show advanced' button". We could even thik about having more of those meta information which could be populated by the binding and can be used if we implement a function for "Create Semantic Model from Inbox in one stop" - which is the main Idea behind #2210

I've created a new Topic and I'm linkid it to this, as my "idea" requires more functions in the core than just having the "Location"