search

steand / optolink

optolink adapter to vitotronic for openhab2
GNU Lesser General Public License v3.0
12 stars 18 forks source link

All set commands do not work with Viessmann KW protocol #10

Closed armintan closed 6 years ago

armintan commented 6 years ago

All set commands do not send the value to be set in ViessmannKW.java

Hotte37 commented 6 years ago

I have the same issues. I found this commit that claims to solve the write problem but it is not merged into the main branch. I am waiting for the new release with this fix since a few month.

@steand Do you see a chance for building a new relase in the near future?

mikeww1 commented 6 years ago

I can confirm that the commit of @Sbried does actually fix the problem. I am using the version with his modifications and there is no USB timeout and I can write in KW mode to my heater. If you know how to compile the stuff you could use the fork of Sbried to get the changes. Of course it would be great if @steand could update his version.

Hotte37 commented 6 years ago

I tried to build the source with luna and oxigen. I allways have trouble with the maven dependencies. Eclipse was not able to find the external packages even if they are downloaded in a local folder and i can even see the reference to it in the project tree of eclipse:( I am new with Java and i read a lot of documentation the last two days ... but obviously not enough. Now i decided to give up.

Was anybody able to build the current version (with the merged pull request #2) and can provide it to @steand to save it as the Release 1.0.1?

Hotte37 commented 6 years ago

I tried to build the jar without maven and this worked.

lewie commented 6 years ago

I hope this helps:

compiled jars: https://github.com/lewie/optolink/tree/master_Sbried_jars changes: https://github.com/steand/optolink/pull/13

Hotte37 commented 6 years ago

Yes it helped. I commended out the "runtime" scope and now it compiles with maven without any problems. I now read the documentation of maven dependency scope and now it is clear to me. But without your help, i never would have found this solution.

steand commented 6 years ago

I have merged: #13

sVnsation commented 6 years ago

I tested the compiled jars of @lewie. The changes of @Sbried breaks the 'set function' for protocol 300.