Nodo-Domotica / Nodo-Core

Arduino based Domotica platform
http://www.nodo-domotica.nl
4 stars 8 forks source link

Plugin_08: Variableset werkt niet voor alle variabelen #907

Open antronics opened 9 years ago

antronics commented 9 years ago

Het commando variableset 7,1 werkt correct echter variableset 3,1 werkt niet. Hierdoor is de waarde van de regenmeter van een WS3500 niet om 00:00 uur te resetten naar 0.

VariableSet 1,10.400 VariableSet 2,70 VariableSet 3,1000002.800 (deze waarde is niet op 0 te zetten) VariableSet 4,0 VariableSet 5,7 VariableSet 6,2.160 VariableSet 7,0

Variable 1t/m6 worden gevuld door plugin 8 Variable 7 door zons op- ondergang

Nodo Domotica controller V3.8 BETA (Mega) (c) Copyright 2015 P.K.Tonkes. Licensed under GNU General Public License. Product=SWACNC-MEGA-R817, ThisUnit=1, ID=LOCNODO

Nodo-Domotica commented 9 years ago

Ik zal er naar kijken. Kun je aangeven met welke build en met de Mega of Small?

Op zo 24 mei 2015 00:01 schreef antronics notifications@github.com:

bijvoorbeeld variableset 7,1 werkt correct echter variableset 3,1 werkt niet.

— Reply to this email directly or view it on GitHub https://github.com/Nodo-Domotica/Nodo-Core/issues/907.

antronics commented 9 years ago

Versie 3.8 en alles draait op een Mega (NES) Product=SWACNC-MEGA-R817 De laatste versie op github staat op 818 en deze is geladen, geeft 817 aan.

Nodo-Domotica commented 9 years ago

Ik kan het niet reproduceren. Kun je aangeven hoe je de variabele set en wanneer het probleem optreedt?

Groeten Paul

Op zo 24 mei 2015 21:55 schreef antronics notifications@github.com:

Versie 3.8 en alles draait op een Mega (NES) Product=SWACNC-MEGA-R817 De laatste versie op github staat op 818 en deze is geladen, geeft 817 aan.

— Reply to this email directly or view it on GitHub https://github.com/Nodo-Domotica/Nodo-Core/issues/907#issuecomment-105057300 .

antronics commented 9 years ago

Dag Paul,

De variabelen 1 t/m 6 worden gevuld door plugin 8, normaliter reset ik variabele 3 vanuit een timer om 00:00 uur naar de waarde 0. Als ik het commando variableset 3,0 ingeef krijg ik een correcte weergave van de events maar de waarde blijft zoals deze was gevuld door de plugin. Ik heb dit voor de variabelen 1t/m 6 op dezelfde wijze getest en allemaal blijven ze de waarde van de plugin aangeven en zijn op de wijze niet te wijzigen. Als ik het commando variableset 7,4 ingeef dan wordt variabele 7 keurig gevuld met de waarde 4, na het ingeven van variableset 7,0 dan gaat de waarde keurig naar 0. Kan het te maken hebben met plugin 8?

Groet Andre

Verstuurd vanaf mijn iPad

Op 25 mei 2015 om 15:08 heeft Paul Tonkes notifications@github.com het volgende geschreven:

Ik kan het niet reproduceren. Kun je aangeven hoe je de variabele set en wanneer het probleem optreedt?

Groeten Paul

Op zo 24 mei 2015 21:55 schreef antronics notifications@github.com:

Versie 3.8 en alles draait op een Mega (NES) Product=SWACNC-MEGA-R817 De laatste versie op github staat op 818 en deze is geladen, geeft 817 aan.

— Reply to this email directly or view it on GitHub https://github.com/Nodo-Domotica/Nodo-Core/issues/907#issuecomment-105057300 .

— Reply to this email directly or view it on GitHub.

Nodo-Domotica commented 9 years ago

Daar kan ik verder mee. Thanks.

Op di 26 mei 2015 01:20 schreef antronics notifications@github.com:

Dag Paul,

De variabelen 1 t/m 6 worden gevuld door plugin 8, normaliter reset ik variabele 3 vanuit een timer om 00:00 uur naar de waarde 0. Als ik het commando variableset 3,0 ingeef krijg ik een correcte weergave van de events maar de waarde blijft zoals deze was gevuld door de plugin. Ik heb dit voor de variabelen 1t/m 6 op dezelfde wijze getest en allemaal blijven ze de waarde van de plugin aangeven en zijn op de wijze niet te wijzigen. Als ik het commando variableset 7,4 ingeef dan wordt variabele 7 keurig gevuld met de waarde 4, na het ingeven van variableset 7,0 dan gaat de waarde keurig naar 0. Kan het te maken hebben met plugin 8?

Groet Andre

Verstuurd vanaf mijn iPad

Op 25 mei 2015 om 15:08 heeft Paul Tonkes notifications@github.com het volgende geschreven:

Ik kan het niet reproduceren. Kun je aangeven hoe je de variabele set en wanneer het probleem optreedt?

Groeten Paul

Op zo 24 mei 2015 21:55 schreef antronics notifications@github.com:

Versie 3.8 en alles draait op een Mega (NES) Product=SWACNC-MEGA-R817 De laatste versie op github staat op 818 en deze is geladen, geeft 817 aan.

— Reply to this email directly or view it on GitHub < https://github.com/Nodo-Domotica/Nodo-Core/issues/907#issuecomment-105057300

.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/Nodo-Domotica/Nodo-Core/issues/907#issuecomment-105333845 .

Nodo-Domotica commented 8 years ago

De plugin voor de regenmeter is de oorzaak. Die moet nog worden aangepast voor Nodo versie 3.8 en hoger. Wellicht lost dit probleem zich op doordat er een Forumgebruiker is die een geheel nieuwe plugin aan het schrijven is. Ik laat dit issue even open staan en label hem als een plugin issue.

Wordt vervolgd.

koningdde commented 8 years ago

De variabele set werkt bij mij wel, inderdaad geeft de plugin een rare waarde terug. Ik kan alleen het verband niet vinden tussen de waarde 999999,0 en de plugin. Deze waarde begint de fout namelijk mee.

De fout lijkt veroorzaakt te worden door plugin.ino, daar is namelijk de waarde terug te vinden:

if defined(PLUGIN_37_COMPATIBILITY)

if(Function==PLUGIN_INIT) for(byte x=0;x<USER_VARIABLES_MAX;x++) UserVar[x]=999999.0; <<<< Hier lijkt de verkeerde waarde vandaan te komen!!

if(Function==PLUGIN_ONCE_A_SECOND) for(byte x=0;x<USER_VARIABLES_MAX;x++) if(UserVar[x]!=999999.0) UserVariableSet(x+1,UserVar[x],false);

Ik heb het helaas nog niet kunnen oplossen....

Groet, Danny

Nodo-Domotica commented 8 years ago

Hoi Danny,

De plugin heeft een aanpassing nodig. De 999999 waarde is een truuc om oude plugins tijdelijk aan het werk te krijgen en moet er uit. Ik zal er morgen eens naar kijken. Proleem is dat dat ik niks kan testen omdat ik de bijbehorende sensor niet heb. Maar ik denk dat we er samen wel uit komen.

Op za 5 sep. 2015 17:45 schreef koningdde notifications@github.com:

De variabele set werkt bij mij wel, inderdaad geeft de plugin een rare waarde terug. Ik kan alleen het verband niet vinden tussen de waarde 999999,0 en de plugin. Deze waarde begint de fout namelijk mee.

De fout lijkt veroorzaakt te worden door plugin.ino, daar is namelijk de waarde terug te vinden:

if defined(PLUGIN_37_COMPATIBILITY)

if(Function==PLUGIN_INIT) for(byte x=0;x<USER_VARIABLES_MAX;x++) UserVar[x]=999999.0; <<<< Hier lijkt de verkeerde waarde vandaan te komen!!

if(Function==PLUGIN_ONCE_A_SECOND) for(byte x=0;x<USER_VARIABLES_MAX;x++) if(UserVar[x]!=999999.0) UserVariableSet(x+1,UserVar[x],false);

Ik heb het helaas nog niet kunnen oplossen....

Groet, Danny

— Reply to this email directly or view it on GitHub https://github.com/Nodo-Domotica/Nodo-Core/issues/907#issuecomment-137969118 .

koningdde commented 8 years ago

Ik help waar ik kan, eventueel testen is ook mogelijk hier. Ik gebruik de V1 van Alecto.

Nodo-Domotica commented 8 years ago

Ik zal de code aanpassen, als jij dan na aanpassingen kan kijken of het werkt zou dat fijn zijn. Want het is voor mij een beetje schieten in het duister omdat ik zelf geen Alecto heb. Wordt vervolgd.

Op zo 6 sep. 2015 21:51 schreef koningdde notifications@github.com:

Ik help waar ik kan, eventueel testen is ook mogelijk hier. Ik gebruik de V1 van Alecto.

— Reply to this email directly or view it on GitHub https://github.com/Nodo-Domotica/Nodo-Core/issues/907#issuecomment-138119124 .

Nodo-Domotica commented 8 years ago

Ik heb de code van Plugin_08 aangepast. Kun je kijken of het werkt?

Groeten Paul.

Nodo-Domotica commented 8 years ago

Overigens, ik zal dit issue renamen, dan is duidelijk dat het om de plugin gaat

koningdde commented 8 years ago

Ik ga hem inladen, even testen wat er gebeurt hier.

koningdde commented 8 years ago

Plugin werkt voor mij nu. Regen komt goed door, alsmede de rest van de waarden. Plugin_37 compatible is nu niet meer nodig.