GUI Generic

[Patryk]

tak naprawdę w kodzie to masz dwa tryby
CONFIG_MODE_10_ON_PRESSES
oraz
CONFIG_MODE_5SEK_HOLD

A jak zmienić, żeby wejście w tryb konfiguracji było po naciśnięciu przycisku przez 5 sekund.
Myślałem, że tak:

Code: Select all

Supla::GUI::addConfigESP(ConfigESP->getGpio(FUNCTION_CFG_BUTTON), ConfigESP->getGpio(FUNCTION_CFG_LED), CONFIG_MODE_10_ON_PRESSES,
                           ConfigESP->getLevel(FUNCTION_CFG_LED));

na

Code: Select all

Supla::GUI::addConfigESP(ConfigESP->getGpio(FUNCTION_CFG_BUTTON), ConfigESP->getGpio(FUNCTION_CFG_LED), CONFIG_MODE_5SEK_HOLD,
                           ConfigESP->getLevel(FUNCTION_CFG_LED));

Ale nie działa.

[PuteR]

przecież są
https://github.com/krycha88/arduino/tre ... pla/sensor

Wybaczcie głupie pytanie ale jak pobrać te wszystkie biblioteki??

[wojtas567]

https://github.com/krycha88/arduino/tree/develop
przycisk code/...

[PuteR]

https://github.com/krycha88/arduino/tree/develop
przycisk code/...

wielkie dzięki -niemogłem tego znaleść

[Goral64]

Ogólnie dobrze by było, aby unikać zmian w samej bibliotece na forkach. Zrobi się z tego bałagan i ludzie się pogubią.
Proponuję działać w taki sposób:
1. Jeśli jest potrzebna jakaś mała, prosta modyfikacja biblioteki - dawajcie znać, mogę na szybko dorzucać potrzebne zmiany. Większe zmiany, które ułatwią dalszą pracę też mogę robić
2. Jeśli wolicie coś zrobić samemu - wrzucajcie pull request do SUPLA/arduino/tree/develop
3. Jesli nie chcecie tego docelowo wrzucać do oficjalnej wersji biblioteki, to nic nie stoi na przeszkodzie, aby zrobić z boku jakieś repo w stylu "SuplaUserExtensions" i tam dorzucać nowe sensory i inne dodatki. Można dać tam folder SuplaExt/sensor/... Wtedy każdy może sobie trzymać dwie biblioteki w Arduino IDE i jedna z drugą nie będzie się kłucić. To powinno działać bez kopiowania biblioteki - tylko pliki z dodatkową implementacją sensorów mogłyby tam być.

Jestem za, ja już przestałem ogarniać kuwetę pod tytułem SuplaDevice Przełączanie się pomiędzy forkami biblioteki pod prawie każdy projekt jest delikatnie mówiąc wkurzające
W wiodącej grupie deweloperów rozwijających funkcjonalności Supli powinno się ustanowić jakiś consensus odnoście pewnych norm i standardów w jakich taki rozwój mógłby się odbywać. Zwłaszcza w kontekście używania bibliotek takich jak SuplaDevice. jeśli ktoś chce rozwijać własną wersję biblioteki powinien zmienić jej nazwę, tak żeby nie kłóciła się z już oficjalnie istniejącą i wszystkie wersje mogły współistnieć we wspólnych zasobach bibliotek Arduino.
Bardzo proszę, abyście jako deweloperzy forkujący SuplaDevice dogadali się w tej kwestii dla dobra nas wszystkich.

[Goral64]

Code: Select all

..\GUI_Generic.ino:114:18: error: 'class SuplaConfigESP' has no member named 'sort'
   if (ConfigESP->sort(FUNCTION_CFG_LED) || ConfigESP->sort(FUNCTION_CFG_BUTTON)) {

Mam aktualną wersję GUI-Generic z repozytorium...

[krycha88]

Code: Select all

..\GUI_Generic.ino:114:18: error: 'class SuplaConfigESP' has no member named 'sort'
   if (ConfigESP->sort(FUNCTION_CFG_LED) || ConfigESP->sort(FUNCTION_CFG_BUTTON)) {

Mam aktualną wersję GUI-Generic z repozytorium...

Który branch?

[Goral64]

Który branch?

Sorry, nie uwzględniłem zmian w pliku GUI-Generic.ino
Teraz mi wyskakuje:

Code: Select all

..\SuplaTemplateBoard.cpp:15:55: error: 'MEMORY_RELAY_RESTORE' was not declared in this scope
       ConfigESP->setGpio(12, 1, FUNCTION_RELAY, HIGH, MEMORY_RELAY_RESTORE);

[krycha88]

Który branch?

Sorry, nie uwzględniłem zmian w pliku GUI-Generic.ino
Teraz mi wyskakuje:

Code: Select all

..\SuplaTemplateBoard.cpp:15:55: error: 'MEMORY_RELAY_RESTORE' was not declared in this scope
       ConfigESP->setGpio(12, 1, FUNCTION_RELAY, HIGH, MEMORY_RELAY_RESTORE);

nie masz dodanego #define SUPLA_BUTTON, pomimo to poprawię to aby się kompilowało

[krycha88]

Jestem za, ja już przestałem ogarniać kuwetę pod tytułem SuplaDevice Przełączanie się pomiędzy forkami biblioteki pod prawie każdy projekt jest delikatnie mówiąc wkurzające
W wiodącej grupie deweloperów rozwijających funkcjonalności Supli powinno się ustanowić jakiś consensus odnoście pewnych norm i standardów w jakich taki rozwój mógłby się odbywać. Zwłaszcza w kontekście używania bibliotek takich jak SuplaDevice. jeśli ktoś chce rozwijać własną wersję biblioteki powinien zmienić jej nazwę, tak żeby nie kłóciła się z już oficjalnie istniejącą i wszystkie wersje mogły współistnieć we wspólnych zasobach bibliotek Arduino.
Bardzo proszę, abyście jako deweloperzy forkujący SuplaDevice dogadali się w tej kwestii dla dobra nas wszystkich.

Jestem ciekawy o jakich problemach mówisz bo wątpię aby było aż tyle zmodyfikowanych bibliotek

wszystkie potrzebne pliki znajdują się na moim forku a do nich zrobiony jest:
https://github.com/SUPLA/arduino/pull/27

Więcej nie da się już zrobić