IndigoDomotics / indigo-yamaharx

Indigo plugin to control Yamaha RX-V receivers
MIT License
5 stars 8 forks source link

Plugin Errors #11

Open alfacruxis opened 7 years ago

alfacruxis commented 7 years ago

I have updated to versión 1.0.2, but upon loading the new plugin, I receive the following errors:

Starting plugin "Yamaha RX Receiver 1.0.2" (pid 2801) Started plugin "Yamaha RX Receiver 1.0.2" Yamaha RX Receiver Error exception in deviceStartComm(Sintoamplificador Living): 'NoneType' object has no attribute 'text' Yamaha RX Receiver Error runConcurrentThread error: Traceback (most recent call last): File "plugin.py", line 190, in runConcurrentThread self.updateStatus(devId) File "plugin.py", line 270, in updateStatus volume = root.find("./Main_Zone/Basic_Status/Vol/Lvl/Val").text AttributeError: 'NoneType' object has no attribute 'text'

Yamaha RX Receiver Error plugin runConcurrentThread function returned or failed (will attempt again in 10 seconds) Yamaha RX Receiver Error runConcurrentThread error: Traceback (most recent call last): File "plugin.py", line 190, in runConcurrentThread self.updateStatus(devId) File "plugin.py", line 270, in updateStatus volume = root.find("./Main_Zone/Basic_Status/Vol/Lvl/Val").text AttributeError: 'NoneType' object has no attribute 'text'

And then it repeats every 10 seconds.

I tried to downgrade to version 1.0.1, but I get the same results. I'm working with Indigo 7.0.3 and a AV Receiver Yamaha RX-V779. The last working plugin version seems to be 0.1.2, which I reverted to.

Any help will be appreciated. Thanks for your time.

indigodomo commented 6 years ago

Unfortunately, that's in the old code (not the receiver code that I added for the vX73 line). Basically what it's saying is that the API call didn't return the volume level. My guess is that the RX-V779 slightly changed the XML protocol.

Looking at the error and the last pre V1 release available in the releases section here (0.1.1), it looks like it should be failing in that release as well. I don't know where you got the 0.1.2 release, perhaps someone fixed it in that one but it's not one of the releases here.

Is the volume actually updating using the version of the plugin you're using? I'd be surprised if it were.

alfacruxis commented 6 years ago

Thanks!

Best regards, Alejandro

El 11 ago. 2017, a las 14:28, indigodomo notifications@github.com escribió:

Unfortunately, that's in the old code (not the receiver code that I added for the vX73 line. Basically what it's saying is that the API call didn't return the volume level. My guess is that the RX-V779 slightly changed the XML protocol.

Looking at the error and the last pre V1 release available in the releases section here (0.1.1), it looks like it should be failing in that release as well. I don't know where you got the 0.1.2 release, perhaps someone fixed it in that one but it's not one of the releases here.

Is the volume actually updating using the version of the plugin you're using? I'd be surprised if it were.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IndigoDomotics/indigo-yamaharx/issues/11#issuecomment-321872028, or mute the thread https://github.com/notifications/unsubscribe-auth/Ab_mEgCfPV65SqpzPGdHdXlPWFEftflCks5sXI8pgaJpZM4N2YN7.

alfacruxis commented 6 years ago

Today tried the latest version of the plug-in (1.0.3) and I'm having the same results:

Yamaha RX Receiver Error exception in deviceStartComm(Sintoamplificador Living): 'NoneType' object has no attribute 'text' Yamaha RX Receiver Error runConcurrentThread error: Traceback (most recent call last): File "plugin.py", line 190, in runConcurrentThread self.updateStatus(devId) File "plugin.py", line 270, in updateStatus volume = root.find("./Main_Zone/Basic_Status/Vol/Lvl/Val").text AttributeError: 'NoneType' object has no attribute 'text'

Yamaha RX Receiver Error plugin runConcurrentThread function returned or failed (will attempt again in 10 seconds)

I returned once again to version 0.1.2 and is working again. Regarding the question of if the v0.1.2 can update the volume, answer is yes, it can. Any directions on how to plot the old code to the new plugin?

Thanks for your time.

indigodomo commented 6 years ago

Zip up the 0.1.2 plugin that works for you and attach it to a reply post here - I'll see if I can figure out what that version did to fix the problem.

alfacruxis commented 6 years ago

Hello,

Sure. Here you have it.

El 26 abr. 2018, a las 14:08, Indigo Domotics notifications@github.com escribió:

Zip up the 0.1.2 plugin that works for you and attach it to a reply post here - I'll see if I can figure out what that version did to fix the problem.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IndigoDomotics/indigo-yamaharx/issues/11#issuecomment-384717351, or mute the thread https://github.com/notifications/unsubscribe-auth/Ab_mEuUOUc1eMW811ebCVIWO_kZbp3HVks5tsf8PgaJpZM4N2YN7.

indigodomo commented 6 years ago

You have to attach it on the actual Issue on Github (https://github.com/IndigoDomotics/indigo-yamaharx/issues/11) - email attachments aren't saved AFAIK.

alfacruxis commented 6 years ago

Oh, sorry. There it comes…

El 26 abr. 2018, a las 14:12, Indigo Domotics notifications@github.com escribió:

You have to attach it on the actual Issue on Github (#11 https://github.com/IndigoDomotics/indigo-yamaharx/issues/11) - email attachments aren't saved AFAIK.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IndigoDomotics/indigo-yamaharx/issues/11#issuecomment-384718724, or mute the thread https://github.com/notifications/unsubscribe-auth/Ab_mEtMh8vZGcs2dLmLfEQ8HQ0NQusDAks5tsgASgaJpZM4N2YN7.

alfacruxis commented 6 years ago

Yamaha RX v0.1.2.zip

indigodomo commented 6 years ago

Someone did some major work on the version you have that was never merged into the master. Turn on debug logging on the 0.1.2 version and watch for any debugging lines in the event log that start with "HTTPError" or "URLError". The version you have has additional error catching code that the newer version doesn't since those changes were never merged into the main plugin.

There is a non-trivial amount of work required to merge the 0.1.2 into the 1.x branch - I'm not going to be able to that any time soon (perhaps someone else would like to make the attempt?). But if there is a specific error that 0.1.2 is catching then I may be able to just catch that error. If you see any of those in the event log then post them here.

alfacruxis commented 6 years ago

Thanks for taking the time to check all that. The version 0.1.2 has no option to turn on/off debugging in the preference setting.

indigodomo commented 6 years ago

It looks like debug is on all the time. Watch for those errors.

alfacruxis commented 6 years ago

Ok. Will post them if they appear. Thanks again!