dev #65

sebastianschwarz · 2022-07-01T05:45:11Z

sebastianschwarz commented

2022-07-01 05:45:11 +00:00

[fix] - Mower with Lona Technologie
[added] - electronic water pump

ab ins svn , pls

[fix] - Mower with Lona Technologie [added] - electronic water pump ab ins svn , pls

marko was assigned by sebastianschwarz

2022-07-01 05:45:12 +00:00

sebastianschwarz added 29 commits 2022-07-01 05:45:13 +00:00

initial bf6734f44b

erste teste 0beab4ec1d

add setter 58333bef34

operatingmode, leakagedetection and turnonpressure fix - ins blaue gerate 43bf81053b

more debug a8646efe0d

add readingss and fix payload b7adf6af6d

vers push 528e8d5a08

fix payload 122983580f

fux payload 0e876c319c

add resetcall + state af2f803be5

fix state 15c791e1bf

test 4e6ca5f8ed

fixing b230baa776

test bd3d354786

test 97b4147072

try 2 b620f87eb1

try 3 33fddad023

revert back d04ea31ef0

fix IODEV d863cbeff0

debug stuff 6490c2f1be

fix lona 0849705c62

lona 9ad11bff63

Merge pull request 'patch_waterpump' (#58 ) from patch_waterpump into dev 234607cb8d

Reviewed-on: #58

add lona readings 40074a3db6

copy & past error 6b475415cc

patch lono. eventuell recrusive readings einbauen. waterpump rechtschreibfehler mit fixed a4a75ab016

Merge pull request 'patch_lona' (#61 ) from patch_lona into dev 768a5a6d7a

Reviewed-on: #61
Reviewed-by: Marko Oldenburg <marko@noreply@cooltux.net>

[fix] - setter procedure e8c2bd5183

Merge pull request '[fix] - setter procedure' (#64 ) from patch_pressure_pump_setlist into dev 41ef5cd10a

Reviewed-on: #64
Reviewed-by: Marko Oldenburg <marko@noreply@cooltux.net>

sebastianschwarz requested review from marko 2022-07-01 05:45:20 +00:00

sebastianschwarz commented

2022-07-01 05:47:45 +00:00

Was heißt denn
"Dieser Branch enthält nicht die neusten Commits der Basis-Branch"

alles patches fließen in dev und dev in testing , was dann ins svn soll

wie kann dann dev oder testing nicht die neusten commits beinhalten? oder liegts daran, dass es patch-xxx branches gibt, die neuer sind auf basis von dev ?

Was heißt denn "Dieser Branch enthält nicht die neusten Commits der Basis-Branch" alles patches fließen in dev und dev in testing , was dann ins svn soll wie kann dann dev oder testing nicht die neusten commits beinhalten? oder liegts daran, dass es patch-xxx branches gibt, die neuer sind auf basis von dev ?

marko commented

2022-07-01 08:11:09 +00:00

Was heißt denn
"Dieser Branch enthält nicht die neusten Commits der Basis-Branch"

alles patches fließen in dev und dev in testing , was dann ins svn soll

wie kann dann dev oder testing nicht die neusten commits beinhalten? oder liegts daran, dass es patch-xxx branches gibt, die neuer sind auf basis von dev ?

Das heist das in der zwischenzeit der dev branch aktuallisiert wurde und dein patch branch nicht mehr aktueller ist wie der dev. Du hättest vorher ein pull vom dev machen müssen bei Dir lokal und den aktuellen dev als basis für einen neuen patch nehmen oder wenn patch schon vorhanden dann ein merge in den patch von dev.

> Was heißt denn > "Dieser Branch enthält nicht die neusten Commits der Basis-Branch" > > alles patches fließen in dev und dev in testing , was dann ins svn soll > > wie kann dann dev oder testing nicht die neusten commits beinhalten? oder liegts daran, dass es patch-xxx branches gibt, die neuer sind auf basis von dev ? > Das heist das in der zwischenzeit der dev branch aktuallisiert wurde und dein patch branch nicht mehr aktueller ist wie der dev. Du hättest vorher ein pull vom dev machen müssen bei Dir lokal und den aktuellen dev als basis für einen neuen patch nehmen oder wenn patch schon vorhanden dann ein merge in den patch von dev.

marko commented

2022-07-01 08:12:15 +00:00

Konntest Du Deine Änderungen testen? Zu mindest so weit das selbst wenn die Änderung nicht funktioniert vorhandene Funktionen erhalten bleiben?

marko commented

2022-07-01 08:15:28 +00:00

Ok sehe gerade das es ja dev nach testing ist. Da war wohl dev nicht aktuell. Ich schaue mal

sebastianschwarz commented

2022-07-01 08:23:41 +00:00

Was heißt denn
"Dieser Branch enthält nicht die neusten Commits der Basis-Branch"

alles patches fließen in dev und dev in testing , was dann ins svn soll

wie kann dann dev oder testing nicht die neusten commits beinhalten? oder liegts daran, dass es patch-xxx branches gibt, die neuer sind auf basis von dev ?

Das heist das in der zwischenzeit der dev branch aktuallisiert wurde und dein patch branch nicht mehr aktueller ist wie der dev. Du hättest vorher ein pull vom dev machen müssen bei Dir lokal und den aktuellen dev als basis für einen neuen patch nehmen oder wenn patch schon vorhanden dann ein merge in den patch von dev.

also ich erstelle alle lokalen patches immer vom origin/dev (remote) , so dass die patches dann auch dorthin ihre commit hash haben.

hatte 3 pathces parallel laufen. aktuell noch den ic24... aber soweit bin ich noch nicht g.

ja der dev wurde ein paar wochen getestet ohne probleme. Zumindest kam kein negativ statement von den testern.

btw

code improvement - eventuell hast du da eine idee

in der GardenaSmartDevice

recrusives abarbeiten

if ( ref( $propertie->{value} ) eq "HASH" ) {
                    while ( my ( $r, $v ) = each %{ $propertie->{value} } ) {
                      if ( ref( $v ) ne "HASH" ) {
                            readingsBulkUpdate(
                              $hash,
                              $decode_json->{abilities}[$abilities]{name} . '-'
                                . $propertie->{name} . '_'
                                . $r,
                                RigReadingsValue( $hash, $v )
                            );
                        } else {
                          while ( my ( $i_r, $i_v ) = each %{ $v } ) {
                            readingsBulkUpdate(
                              $hash,
                              $decode_json->{abilities}[$abilities]{name} . '-'
                              . $propertie->{name} . '_'
                              . $r . '_' . $i_r,
                              RigReadingsValue( $hash, $i_v )
                            );
                          }
                        }

der else zweig ist neu. da es hier eine verschachtelung tiefer gibt , seit den lonas ...

> > Was heißt denn > > "Dieser Branch enthält nicht die neusten Commits der Basis-Branch" > > > > alles patches fließen in dev und dev in testing , was dann ins svn soll > > > > wie kann dann dev oder testing nicht die neusten commits beinhalten? oder liegts daran, dass es patch-xxx branches gibt, die neuer sind auf basis von dev ? > > > > Das heist das in der zwischenzeit der dev branch aktuallisiert wurde und dein patch branch nicht mehr aktueller ist wie der dev. Du hättest vorher ein pull vom dev machen müssen bei Dir lokal und den aktuellen dev als basis für einen neuen patch nehmen oder wenn patch schon vorhanden dann ein merge in den patch von dev. also ich erstelle alle lokalen patches immer vom origin/dev (remote) , so dass die patches dann auch dorthin ihre commit hash haben. hatte 3 pathces parallel laufen. aktuell noch den ic24... aber soweit bin ich noch nicht *g*. ja der dev wurde ein paar wochen getestet ohne probleme. Zumindest kam kein negativ statement von den testern. btw code improvement - eventuell hast du da eine idee in der GardenaSmartDevice recrusives abarbeiten ``` if ( ref( $propertie->{value} ) eq "HASH" ) { while ( my ( $r, $v ) = each %{ $propertie->{value} } ) { if ( ref( $v ) ne "HASH" ) { readingsBulkUpdate( $hash, $decode_json->{abilities}[$abilities]{name} . '-' . $propertie->{name} . '_' . $r, RigReadingsValue( $hash, $v ) ); } else { while ( my ( $i_r, $i_v ) = each %{ $v } ) { readingsBulkUpdate( $hash, $decode_json->{abilities}[$abilities]{name} . '-' . $propertie->{name} . '_' . $r . '_' . $i_r, RigReadingsValue( $hash, $i_v ) ); } } ``` der else zweig ist neu. da es hier eine verschachtelung tiefer gibt , seit den lonas ...

marko added 2 commits 2022-07-01 08:28:14 +00:00

Merge pull request 'dev' (#54 ) from dev into main 8afc912ed5