GameDevHobby / Domoticz-LGTV-WebOS-Plugin

LG TV WebOS 3 Plugin for Domoticz
14 stars 13 forks source link

issue with 2 LG TV #8

Open chtidav opened 6 years ago

chtidav commented 6 years ago

Hello,

First thank you for your great job for the plugin !

It seems that you do not have implemented multiple LG TV : cause when I accept on the first TV the connexion from the plugin, the second one ask me for to accept connexion and when I accpet it in the second, it's the same with the first :( Maybe it could be simple to add this feature by testing the id of devices with the help of a list when updating status. Could you help me to try to add this feature or do you think you will implement it ? Thanks in advance, /Dav

wmmudde commented 6 years ago

Hi friend,

Can you please tell me how you installed the plugin in domoticz and which one? i cant get it working..

GameDevHobby commented 6 years ago

Feel free to make a pull request with the features in it. I only have one lg tv so I wouldn't be able too test it. Have you tried adding this plugin twice and having two pieces of hardware in domoticz? I'm not sure that would work. But that might be easier to get working if it doesn't.

chtidav commented 6 years ago

yes I tried that : paste with different name two times the plugin in plugins dir, configured 2 hardware with each plugin dedicated... but have same issue : seems that the token exchanged between TV and plugin are store in the same memory allocation although with 2 TV we should store each token in a array... that's why I have to re-associate everytime I switch of TV... Not sure where to place that in the script :(

GameDevHobby commented 5 years ago

Does the hidden file ".pylgtv" in your home directory contain 2 entries? one for each IP of the tvs?

Floppik commented 5 years ago

Same issue. If i wrote ip of each tv on .pylgtv - all tv stop working with plugin. How run this plugin on 2 tv s simulationly?

kitopopo commented 3 years ago

same problem. any solution? thanks in advanced

SkeletonRocky commented 3 years ago

Hey Guys, just had the same issue. Finally figured it out.

On the lg.py file, modify the the firsh section from this: class LgCommand (object): def __init__(self, ip): self.client = WebOsClient(ip)

to this: class LgCommand (object): def __init__(self, ip): KEYPATH = '.pylgtv' self.client = WebOsClient(ip,KEYPATH+ip)

it will create separated files with the auth token from TV, and you can use the same plugin for all of your TVs.

kitopopo commented 3 years ago

Oh yeah, thanks very much. I will test the plugin in 13 days. I will inform. Thanks again. Best regards

kitopopo commented 2 years ago

Hey Guys, just had the same issue. Finally figured it out.

On the lg.py file, modify the the firsh section from this: class LgCommand (object): def __init__(self, ip): self.client = WebOsClient(ip)

to this: class LgCommand (object): def __init__(self, ip): KEYPATH = '.pylgtv' self.client = WebOsClient(ip,KEYPATH+ip)

it will create separated files with the auth token from TV, and you can use the same plugin for all of your TVs.

Dear friend, it doesn't work. After modify lg.py appears the followings errors:

line 16 self commandlines indentation error unexpected indent.

No works my first lg tv . Any idea? thanks in advanced

SkeletonRocky commented 2 years ago

Are you sure, there is no typo in it? This is my lg.py fist block: https://pastebin.com/KHUyfT3c! mtputty_m8LGU6VSLm

kitopopo commented 2 years ago

Are you sure, there is no typo in it? This is my lg.py fist block: https://pastebin.com/KHUyfT3c! mtputty_m8LGU6VSLm

Thanks for share your file. Its works with 2 lgtv. thanks dear friend. It would be interesting if the plugin developer included this modification in the original file. Kind regards

SkeletonRocky commented 2 years ago

I have already committed a pull request.

Sent from my iPhone

On 2021. Sep 11., at 15:19, kitopopo @.***> wrote:



Are you sure, there is no typo in it? This is my lg.py fist block: https://pastebin.com/KHUyfT3c!https://pastebin.com/KHUyfT3c mtputty_m8LGU6VSLmhttps://user-images.githubusercontent.com/33547046/132948229-99096e99-a871-4f86-93f4-bf664a4fd3f7.png

Thanks for share your file. Its works with 2 lgtv. thanks dear friend. It would be interesting if the plugin developer included this modification in the original file. Kind regards

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/GameDevHobby/Domoticz-LGTV-WebOS-Plugin/issues/8#issuecomment-917406474, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH76GJXPE4JK4HO6EQUVOFDUBNJMNANCNFSM4EKI6YXQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

kitopopo commented 2 years ago

I have already committed a pull request. Sent from my iPhone On 2021. Sep 11., at 15:19, kitopopo @.***> wrote:  Are you sure, there is no typo in it? This is my lg.py fist block: https://pastebin.com/KHUyfT3c!https://pastebin.com/KHUyfT3c mtputty_m8LGU6VSLmhttps://user-images.githubusercontent.com/33547046/132948229-99096e99-a871-4f86-93f4-bf664a4fd3f7.png Thanks for share your file. Its works with 2 lgtv. thanks dear friend. It would be interesting if the plugin developer included this modification in the original file. Kind regards — You are receiving this because you commented. Reply to this email directly, view it on GitHub<#8 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH76GJXPE4JK4HO6EQUVOFDUBNJMNANCNFSM4EKI6YXQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Dear friend

Hey Guys, just had the same issue. Finally figured it out. On the lg.py file, modify the the firsh section from this: class LgCommand (object): def __init__(self, ip): self.client = WebOsClient(ip) to this: class LgCommand (object): def __init__(self, ip): KEYPATH = '.pylgtv' self.client = WebOsClient(ip,KEYPATH+ip) it will create separated files with the auth token from TV, and you can use the same plugin for all of your TVs.

Dear friend, it doesn't work. After modify lg.py appears the followings errors:

line 16 self commandlines indentation error unexpected indent.

No works my first lg tv . Any idea? thanks in advanced

Dear friend,

after modify lg.py I obtained some errors in domoticz with two lg tv, also this erros can be seen in the widgets of domoticz app. I also observed a slowdown of the zigbee devices in domoticz, did the same thing happen to you?

thanks in advanced.

Best regards

SkeletonRocky commented 2 years ago

Nope. I am using the same solution what I have wrote earlier, still with the same 2 TVs and its still working properly.

kitopopo commented 2 years ago

Hey Guys, just had the same issue. Finally figured it out.

On the lg.py file, modify the the firsh section from this: class LgCommand (object): def __init__(self, ip): self.client = WebOsClient(ip)

to this: class LgCommand (object): def __init__(self, ip): KEYPATH = '.pylgtv' self.client = WebOsClient(ip,KEYPATH+ip)

it will create separated files with the auth token from TV, and you can use the same plugin for all of your TVs.

Hi dear friend,

In my particular case and after many attempts this script modification does not work correctly when the two LG TV's are turned on at the same time, various problems occur, I cannot act on the TV's and domoticz crashes. Let's hope that the developer @GameDevHobby or any other user with knowledge of scripts can modify the original script since it would be interesting to have more than 1 TV. Anyway, I appreciate your modification and I'm glad it works for you. A cordial greeting.