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

Smartthings Integration Refrigerator temperature and set point unknown #61075

Open
gwheels opened this issue Dec 6, 2021 · 60 comments
Open

Smartthings Integration Refrigerator temperature and set point unknown #61075

gwheels opened this issue Dec 6, 2021 · 60 comments

Comments

@gwheels
Copy link

gwheels commented Dec 6, 2021

The problem

New Refrigerator with refrigerator and freezer temperature and setpoints showing in Smartthings, but show as "unknown" in smartthings integration.
Screenshot 2021-12-05 220341

What version of Home Assistant Core has the issue?

core-2021.11.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Smartthings

Link to integration documentation on our website

https://www.home-assistant.io/integrations/smartthings

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@probot-home-assistant
Copy link

smartthings documentation
smartthings source
(message by IssueLinks)

@probot-home-assistant
Copy link

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

@firstof9
Copy link
Contributor

firstof9 commented Dec 8, 2021

This happens because the smartthings API responds with:

        "main": {
...
            "temperatureMeasurement": {
                "temperature": {
                    "value": null
                }
            },
...

The measurement and setPoint have moved under the "cooler" key instead:

        "cooler": {
            "contactSensor": {
                "contact": {
                    "value": "closed",
                    "timestamp": "2021-12-08T18:13:24.859Z"
                }
            },
            "custom.disabledCapabilities": {
                "disabledCapabilities": {
                    "value": [],
                    "timestamp": "2021-07-27T01:19:43.145Z"
                }
            },
            "temperatureMeasurement": {
                "temperature": {
                    "value": 37,
                    "unit": "F",
                    "timestamp": "2021-12-02T16:11:13.294Z"
                }
            },
            "custom.thermostatSetpointControl": {
                "minimumSetpoint": {
                    "value": 34,
                    "unit": "F",
                    "timestamp": "2021-07-27T01:19:42.452Z"
                },
                "maximumSetpoint": {
                    "value": 44,
                    "unit": "F",
                    "timestamp": "2021-07-27T01:19:42.452Z"
                }
            },
            "thermostatCoolingSetpoint": {
                "coolingSetpoint": {
                    "value": 37,
                    "unit": "F",
                    "timestamp": "2021-07-27T01:19:42.452Z"
                }
            }
        },

Similar with the freezer info:

        "freezer": {
            "contactSensor": {
                "contact": {
                    "value": "closed",
                    "timestamp": "2021-12-08T07:17:33.154Z"
                }
            },
            "custom.disabledCapabilities": {
                "disabledCapabilities": {
                    "value": [],
                    "timestamp": "2021-07-27T01:19:43.145Z"
                }
            },
            "temperatureMeasurement": {
                "temperature": {
                    "value": 0,
                    "unit": "F",
                    "timestamp": "2021-11-13T23:31:15.088Z"
                }
            },
            "custom.thermostatSetpointControl": {
                "minimumSetpoint": {
                    "value": -8,
                    "unit": "F",
                    "timestamp": "2021-07-27T01:19:42.452Z"
                },
                "maximumSetpoint": {
                    "value": 5,
                    "unit": "F",
                    "timestamp": "2021-07-27T01:19:42.452Z"
                }
            },
            "thermostatCoolingSetpoint": {
                "coolingSetpoint": {
                    "value": 0,
                    "unit": "F",
                    "timestamp": "2021-07-27T01:19:42.452Z"
                }
            }
        },

So in order for this to work again I think the library would need updating to allow enumeration of all the keys under "components".

@gwheels
Copy link
Author

gwheels commented Dec 24, 2021

Thanks. Which library needs the update, and who manages it?

@firstof9
Copy link
Contributor

Library is pysmartthings, keep in mind I'm not 100% sure that's where the issue is located.

@coffeexclutter
Copy link

I'm in the same boat. I would love to see an update on this one.

@kernelpanic85
Copy link
Contributor

I had the same issue and found that going into the Smartthings IDE and setting the type to Samsung OCF Refrigerator on the parent device and then Samsung OCF Refrigerator Compartment on the child devices seemed to fix it. I still have an issue with HA not reporting all the sensor values that are present from the Smartthings API, but I suppose that is a separate issue.

@gwheels
Copy link
Author

gwheels commented Mar 24, 2022

That did the trick! once I determined that the Smartthings IDE is on the Samsung smartthings website. I can now see the refrigerator temperature and setpoint.

How did you configure the freezer child device? I have set the "Refrigerator Freezer" and "freezer" child devices as Samsung OCF Refrigerator Compartment type and tried both the same and unique device network ID's but the temperature and setpoint are not passing to HA like the refrigerator compartment.

@kernelpanic85
Copy link
Contributor

Yes, that seems like a separate issue. I found a bunch of closed issues on Github regarding this and they seem to say that the integration currently doesn't enumerate all the info from the child devices so we are missing all of those sensors. Maybe we can re-surface this and hopefully someone can take a look?

@gwheels
Copy link
Author

gwheels commented Mar 24, 2022

Hopefully so. Still, some progress is good.

@kernelpanic85
Copy link
Contributor

@andrewsayre any hope of getting the full child components into the integration for the refrigerators?

@markdepalma
Copy link

I had the same issue and found that going into the Smartthings IDE and setting the type to Samsung OCF Refrigerator on the parent device and then Samsung OCF Refrigerator Compartment on the child devices seemed to fix it. I still have an issue with HA not reporting all the sensor values that are present from the Smartthings API, but I suppose that is a separate issue.

This worked for me too!

@firstof9
Copy link
Contributor

any hope of getting the full child components into the integration for the refrigerators?

You may run into API limitations if you attempt to subscribe to too many entries. Currently the SmartThings API limits you to 40 subscriptions per app. Not sure why 40, seems like weird limit myself.

@markdepalma
Copy link

any hope of getting the full child components into the integration for the refrigerators?

You may run into API limitations if you attempt to subscribe to too many entries. Currently the SmartThings API limits you to 40 subscriptions per app. Not sure why 40, seems like weird limit myself.

Wondering the same here. I have the base fridge/energy stats working but freezer is a child object and not present. I could create my own integration if I have to...

@firstof9
Copy link
Contributor

I could create my own integration if I have to...

It's been done, I'm sure the author wouldn't mind some additional collaboration.

https://github.com/veista/smartthings

@markdepalma
Copy link

I could create my own integration if I have to...

It's been done, I'm sure the author wouldn't mind some additional collaboration.

https://github.com/veista/smartthings

Ah, I didn't know about this. Will check it out later. Thanks!

@markdepalma
Copy link

markdepalma commented May 25, 2022

Just added on support for my fridge using the work already done by you. Created a pull request: veista/smartthings#23.

@github-actions
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Aug 27, 2022
@gwheels
Copy link
Author

gwheels commented Aug 27, 2022

Stale? Seems so harsh. Still haven't been able to work through this.

How does this look for freezer settings IDE? Obviously temp and setpoint there, but not passing to the HA integration.

ascrnsht

@janivr
Copy link

janivr commented Sep 18, 2022

Any fix to not show freezer data, but only cooler?
Any workaround I can use?

@janivr
Copy link

janivr commented Sep 18, 2022

Just added on support for my fridge using the work already done by you. Created a pull request: veista/smartthings#23.

Installed this but got only 4 entities, cooler temp is in but not freezer temp.

@coffeexclutter
Copy link

coffeexclutter commented Oct 11, 2022 via email

@kernelpanic85
Copy link
Contributor

I think I added a Device Network ID to mine. It let me change it no problem. My device type is Samsung OCF Refrigerator.

@gwheels
Copy link
Author

gwheels commented Oct 14, 2022

I think I added a Device Network ID to mine. It let me change it no problem. My device type is Samsung OCF Refrigerator.

When you add the Device Network ID, does it push the freezer child device to the Smartthings Integration? I've tried both a unique or the same Network ID number as the parent refrigerator ID. Neither causes a new entity in the integration. The freezer temp is present in the native smartthings app (as is my deli drawer), so I know it's there on the Smartthings side.

@bearhntr
Copy link

bearhntr commented Nov 7, 2022

I am trying to follow this - and not sure things are working. I did Uninstall the SmartThings - followed the steps for putting the HACS version - and I am getting a little more -- but still not sure what is wrong. I would really like to get this working - as I hate having to use the ALEXA Link to SmartThings. I also have a RANGE to get installed, a DISHWASHER and a RANGE-HOOD MICROWAVE (soon as my new floors get installed. Wondering if they are all going to give me fits too. Seems there is no way to update Firmware or Software on these things like the Family Hub Fridge does on its own.

image

@bearhntr
Copy link

bearhntr commented Nov 7, 2022

The WASHER-- definately shows more information - but you have to turn it ON to get more. Been my complaint to SAMSUNG for 1.5 years -- they are Supposed to be SMART, why do I have to walk to it to turn it ON?

image

and the DRYER

image

@bearhntr
Copy link

bearhntr commented Nov 7, 2022

What I see in the ENTITIES

image

@scfigg
Copy link

scfigg commented Jan 6, 2023

Just added on support for my fridge using the work already done by you. Created a pull request: veista/smartthings#23.

Installed this but got only 4 entities, cooler temp is in but not freezer temp.

yeah I installed the Custom integration and still not seeing the fridge temp (new 2022 family Hub) image

Same. Tried changing "type" of each sensor in SmartThings API device settings, which provided temp data to display in SmartThings (in pic below), but not in ST Integrations. image

I just installed this integration with our new fridge. Same issue here. Ever end up figuring out a fix?

@atkjedi
Copy link

atkjedi commented Jan 6, 2023

I was able to get values by adding in my model manually to the custom integration editing the following files
Custom components/smartthings
(binary_sensor.py, number.py, select.py, sensor.py and switch.py)

My model number was 22K_REF_LCD_FHUB7

Got the info on this from here
veista/smartthings#22

Screenshot_20230106_005442_Home Assistant

Note: I also had to update the contact sensors to look for close vs closed

veista/smartthings#32 (comment)

@Swallowtail23
Copy link

@atkjedi you have the same fridge as me, a 22K_REF_LCD_FHUB7. I notice your temperature is coming through correctly (37F). Mine comes through as 20C, even though the API says 3C. The setpoint says 3C, the fridge itself says it's at 3C, SmartThings API and app say it's at 3C... but this integration says it's at 20C.

@Particpant
Copy link

That's cool, but why can't the standard smart things integration get that stuff?
Isn't the point of having a smart things integration that it's the only way to interface with samsung appliances?

@kernelpanic85
Copy link
Contributor

I was able to get values by adding in my model manually to the custom integration editing the following files Custom components/smartthings (binary_sensor.py, number.py, select.py, sensor.py and switch.py)

My model number was 22K_REF_LCD_FHUB7

Got the info on this from here veista/smartthings#22

Screenshot_20230106_005442_Home Assistant

Note: I also had to update the contact sensors to look for close vs closed

veista/smartthings#32 (comment)

I'm still having trouble with my fridge. Could you screenshot what your fridge looks like in the Smartthings IDE? What parent and child devices show up and what do you have their type set to?

@atkjedi
Copy link

atkjedi commented Apr 21, 2023

Screenshot_20230421_140900_Chrome

Sure, here it is in the IDE, but I did not make any changes because of the way things are now handles in Smart things. All of the child objects are also placeholder types.
In fact I removed and re-added my fridge so it has the default settings because changes I made in the IDE caused problems (that I can't remember now)

@gwheels
Copy link
Author

gwheels commented Apr 22, 2023

Loading the veista smartthings custom integration and adding my model number (TP2X_REF_20K) from the smartthings IDE into the Custom components/smartthings files binary_sensor.py, number.py, select.py, sensor.py and switch.py did the trick. Thanks all.
ascrnsht

@kernelpanic85
Copy link
Contributor

Screenshot_20230421_140900_Chrome

Sure, here it is in the IDE, but I did not make any changes because of the way things are now handles in Smart things. All of the child objects are also placeholder types. In fact I removed and re-added my fridge so it has the default settings because changes I made in the IDE caused problems (that I can't remember now)

Loading the veista smartthings custom integration and adding my model number (TP2X_REF_20K) from the smartthings IDE into the Custom components/smartthings files binary_sensor.py, number.py, select.py, sensor.py and switch.py did the trick. Thanks all. ascrnsht

Thanks guys for the screenshots. Perhaps my TP6X_REF_16K fridge is just not as smart, or requires more changes somewhere else like the pysmartthings library or something. I'm not really sure. It's not the family hub model with the screen, but I still see lots of data in the Smartthings IDE that is not showing up in HA at all like your fridges.

image
image

@firstof9
Copy link
Contributor

Perhaps my TP6X_REF_16K fridge is just not as smart

Add your model number to the files listed in this PR

@kernelpanic85
Copy link
Contributor

Perhaps my TP6X_REF_16K fridge is just not as smart

Add your model number to the files listed in this PR

I can, but I made those changes locally and they did not make any difference. This is all I've got with those changes in place:

image

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@jhhoffma3
Copy link

I don't know that it's stale, but there's certainly something wrong without with the addon or (more likely) the SmartThings API.

My setup and integrations all became unavailable some time ago (months?). First my Fridge basically lost everything overnight and then, more recently, my washer and dryer are "unavailable".

I did notice that the website that used to take you to the API is different and then old site is no longer available.

@bearhntr
Copy link

@jhhoffma3

I completely agree. I never got it to work. It would pull in only 'parts' of the washer, the dryer, and the fridge was completely useless.

@jhhoffma3
Copy link

I played around with this a little today and found that I had to go into the SmartThings app and re-allow permissions for the "automation". That brought back the availability for most of my devices.

My Washer/Dryer are fully back up, but only some of the Refrigerator options are showing up. Missing in action are the actual control elements (temps, feature on/off).

Screenshot 2023-07-24 9 43 25 PM

@atkjedi
Copy link

atkjedi commented Jul 25, 2023

Mine is now only showing a few basic sensors

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@jhhoffma3
Copy link

This is still an issue. Many sensors and functions are no longer available.

@jhhoffma3
Copy link

Does anyone know how to uninstall this addon and go back to the default integration?

@contemplator1998
Copy link

Just in case, here is a fixed (at least for me) SmartThings you can install via HACS: GitHub - contemplator1998/smartthings

(Pulled #99924 and wrapped into a HACS repo)

It makes everything accessible, even sensors/controls that are always unavailable (just disable them in the HA).

@jhhoffma3
Copy link

Just in case, here is a fixed (at least for me) SmartThings you can install via HACS: GitHub - contemplator1998/smartthings

(Pulled #99924 and wrapped into a HACS repo)

It makes everything accessible, even sensors/controls that are always unavailable (just disable them in the HA).

Are you claiming this update fixes the "devices unavailable issue" or that it merely shows all entities, regardless of their availability status to HA?

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@Particpant
Copy link

still not resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.