search

mookfist / repo

Kodi Repository
6 stars 4 forks source link

Not working for me #29

Closed exxeraga closed 7 years ago

exxeraga commented 7 years ago

Lights do not react at all if I set host and port manually, scan gives me an error. Here's what I found in log file. Any ideas? Thank for your work!

16:30:06 T:3529503648 WARNING: CPythonInvoker(31): Script invoked without an addon. Adding all addon modules installed to python path as fallback. This behaviour will be removed in future version. 16:30:17 T:3529503648 ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--

mookfist commented 7 years ago

Thank you for the bug report.

I feel like there are two problems here:

  1. For the scanning bit, if you have an Android phone, can you try installing this app: https://play.google.com/store/apps/details?id=com.lierda.wifi&hl=en and see if it can detect your bridge automatically? I'm not sure what an iOS equivalent app is but i'm sure one exists. It seems that it can't detect your bridge, but then the plugin doesn't handle this situation gracefully. But I do want to make sure you can verify that something else which is known to work can find your wifi bridge.

  2. The second error at the bottom of this log seems to suggest that the plugin is not fully configured. Again, this should be a situation that the plugin handles gracefully. I should probably make sure there are sensible default values for all the settings in the plugin to be safe.

exxeraga commented 7 years ago

That version of app doesnt work for me, maybe a problem with android 7 or with the bridge, cant be sure. I installed mi-light 3 from play store and its working, bridge gets discovered. About plugin configuration, I tryed auto discovery and when that failed I gave it IP address and port manually (8899). I have this bridge : http://www.milight.com/milight-wifi-receiver-bridge-3-0-controller-box/

mookfist commented 7 years ago

Can you visit the wifi bridge's webpage and let me know what the software version is? This screenshot is from my bridge:

wifi-bridge-settings

exxeraga commented 7 years ago

Sure, here it is.

ver

trarizakaria commented 7 years ago

I'm in the same situation. Same bridge. But android app works with me

trarizakaria commented 7 years ago

screenshot_2017-03-04-07-00-03 I tested the addon on kodi 17 and 15.2 same problem. I think there is something with the bridge v6

exxeraga commented 7 years ago

Any ideas?

mookfist commented 7 years ago

so you guys appear to be running a different version of the wifi bridge than I am. I have the v4 firmware.

AT this time I don't plan to support newer versions. This is mainly because I am relying on a third party library that has implemented the LimitlessLED protocol and I don't want to implement the whole limitlessled protocol myself.

I'll purchase a modern bridge soon, and keep my eyes peeled for updates to the library or a new python library that implements the newer standards.

trarizakaria commented 7 years ago

Thank you

mookfist commented 7 years ago

I have ordered the v6 wifi bridge. It should take two weeks to arrive so it'll be some time before I get v6 working

trarizakaria commented 7 years ago

Good news :) i have kodi running on raspberry (raspbian), there is a way to have an ambilight on raspberry (may be with an arduino) ??

mookfist commented 7 years ago

ha, ambilight type functionality is well beyond the scope of this project.

I'm not sure if there are other Kodi projects for this type of functionality.

On Sun, Apr 2, 2017 at 4:30 AM, trarizakaria notifications@github.com wrote:

Good news :) i have kodi running on raspberry (raspbian), there is a way to have an ambilight on raspberry (may be with an arduino) ??

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/mookfist/repo/issues/29#issuecomment-290972548, or mute the thread https://github.com/notifications/unsubscribe-auth/AOYnNEWwwn3Ckff5QGGvP65hKnY7wFByks5rr1ydgaJpZM4MRO69 .

mookfist commented 7 years ago

the bridge has been shipped!

I've decided to implement the protocol myself in the end. A python library is now available at https://github.com/mookfist/mookfist-lled-controller - I will use this with the kodi plugin and since I'll have a v4 and a v6 bridge I should be able to test against both and support both.

mookfist commented 7 years ago

For those interested, I will be getting the new wifi bridge the day after this comment was written. I am going to implement the protocol myself in the end.

However, I've started a new 0.2.0 branch of this plugin, which is where I will implement these changes. I've started this branch because it's more or less a rewrite of how the plugin talks to the wifi bridge. I'm doing this out of necessity as currently this plugin can struggle sometimes when trying to fade slowly and based on my testing, the approach I took is problematic.

So it may still be a while yet before I can support the v6 bridge, but it is being worked on!

trarizakaria commented 7 years ago

We will wait. Take your time. Good luck

exxeraga commented 7 years ago

Thank you for your work. Looking forward to it!

mookfist commented 7 years ago

well on the library i'm developing for this, I got my v6 bridge working. If anyone is up for it try it out: https://github.com/mookfist/mookfist-lled-controller. I now need to integrate this into the kodi plugin

mookfist commented 7 years ago

most things appear to be working now. I will do some more testing but I think I'm pretty close to pushing the new version of this plugin to the master branch

mookfist commented 7 years ago

Hello everyone, the plugin seems to be working for me now. I wlil continue to use it and fix anything that comes up. If anyone wants to, please feel free to try out the beta yourself by installing the development repository at https://github.com/mookfist/repo/raw/0.2.0devel/zips/repository.mookfist/repository.mookfist-2.2.zip

the plugin has a very different way of being configured now with more tweaking options and per-group settings. You can review the setup instructions here: https://github.com/mookfist/repo/tree/0.2.0devel/script.service.mookfist-milights

Please let me know if you have any issues. I'd say a few more weeks and i'll push this into master.

exxeraga commented 7 years ago

Ok, I've installed new version. This time bridge is detected but still cant get it to work on test or anywhere else after detection (I've saved configuration after it was detected as you suggested). I've tried running it on amlogic media player running libreelec and tried it on windows pc, same problem and same errors in log.

`20:30:08.252 T:3966759840 ERROR: [mookfist-milights::Controller] Bridge not configured properly

20:30:12.395 T:3966759840 ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--

mookfist commented 7 years ago

hrm...

can you confirm you have beta2 version installed?

The issue seems to be when it tries to parse the color. I will try setting the default color to white so that there is always a color value present.

I will make this change tonight (EST) and then the version will be beta3

mookfist commented 7 years ago

actually scratch that, the better solution should just make sure a color is set before trying to parse anything.

mookfist commented 7 years ago

version 0.2.0~beta4 is up. New icon too and should resolve your issue @exxeraga

Since this ticket seemed to be originally for ver6 support, I will now now close it.

Please feel free to open new issues for any additional issues you may encounter. Thank you all for your patience!

exxeraga commented 7 years ago

Great, I'll test it tonight. Thank you very much!!!

exxeraga commented 7 years ago

No luck for me, still getting ERROR: [mookfist-milights::Controller] Bridge not configured properly... It works great with android app and limitless led windows app. Not sure why it wont work with this plugin :(