home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
70.07k stars 29.14k forks source link

Error when trying to assign a unique id to noaa_tides #119319

Open jlipsit opened 1 month ago

jlipsit commented 1 month ago

The problem

An error is generated when you try to assign a uniuqe ID to noaa_tides. A unique ID is required so that "Assist" will be aware of the tide information so that it can be exposed to AI for easy access.

# Tide Sensor
- platform: noaa_tides
  station_id: 8722494
  unique_id: jupiter_tide

ERROR: Invalid config for 'noaa_tides' from integration 'sensor' at sensors.yaml, line 7: 'unique_id' is an invalid option for 'sensor.noaa_tides', check: unique_id

In order to expose the sensor data to "Assist" it needs a unique ID. image

Once the data is exposed to assist you can use AI (CharGPT) to read sensor data to make the house of the future: image

No tide information. :-( image

What version of Home Assistant Core has the issue?

2024.6.1

What was the last working version of Home Assistant Core?

None

What type of installation are you running?

Home Assistant OS

Integration causing the issue

noaa_tides

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

# Tide Sensor
- platform: noaa_tides
  station_id: 8722494
  unique_id: jupiter_tide

Anything in the logs that might be useful for us?

No response

Additional information

Core 2024.6.1 Supervisor 2024.06.0 Operating System 12.3 Frontend 20240605.0

home-assistant[bot] commented 1 month ago

Hey there @jdelaney72, mind taking a look at this issue as it has been labeled with an integration (noaa_tides) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `noaa_tides` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign noaa_tides` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


noaa_tides documentation noaa_tides source (message by IssueLinks)

knuckleheadsmiff commented 4 weeks ago

I just ran into it not having a unique_id for noaa_tides and getting the same error. I didn’t bother to try adding my own unique_id because the way I read the doc I can’t.

I did this to see if I could add units to the display of the tide heights (they are in feet but don’t display the unit.)