Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem z zapisem ustawień #85

Open
krzaknet opened this issue Feb 15, 2020 · 13 comments
Open

Problem z zapisem ustawień #85

krzaknet opened this issue Feb 15, 2020 · 13 comments
Labels

Comments

@krzaknet
Copy link

Wersja 2.7.5

Po wyborze i zapisaniu ustawień (czujnik zanieczyszczeń - Sensirion SPS30 ) nie zapamiętuje ustawień.

image

Testowane na ostatnim Safari oraz Chrome

@bfaliszek
Copy link
Member

Samemu kompilowałeś kod, czy wgrałeś gotowy .bin?

@krzaknet
Copy link
Author

Gotowy .bin

@krzaknet
Copy link
Author

Chwilowo udało mi sie obejsc problem, kompilujac samodzielnie .bin ze zmienionym defaultowym configiem.

@bfaliszek
Copy link
Member

A co w nim zmieniłeś?

@bfaliszek
Copy link
Member

Problem z zapisem konfiguracji wynika raczej z problemów z zapisem danych do pamięci urządzenia(SPIFFS) niż zmianami w konfiguracji. defaultConfig używany jest tylko jeśli nie można stworzyć/odczytać pliku config.json, który jest w SPIFFS.

@leventehuszar
Copy link

Hi,
I have so this problem, too.
I installed sps30 sensor and when I would setup the dust sensor type on the interface site, the esp droped the sensors.
I used the original .bin. I probed the last and the previous version, too. Both firmware do it this.

@krupniok200
Copy link

Można powiedzieć że też mam podobny problem, ja kompiluje sam już z spreparowanym configiem, po wgraniu .bin nie chce wgl robić pomiarów, lecz pomaga zmiana ustawienia (ja np włączam debuga, to jest pewnie w 100% że zadziała), w ostatniej kompilacji zmiana ustawień powoduje usunięcie i wyłączenie przesyłania do aqi, ponowne zapisanie ustawień z aqi kończy się fiaskiem

@krzaknet
Copy link
Author

@bfaliszek

Zmieniałem defaultowy config (DUST_MODEL) po problemach z zapisywaniem ustawień konfiguracji z oficjalnej .bin

Problem dotyczy pola pm 2.5 sensor w interfejsie web - po wybraniu i zapisaniu konfiguracji zawsze wraca pole defaultowe "without sensor"

Problem ten np. nie dotyczy na przykład pola wyżej - "Temp / Hum".

Wszystko robione wg. tutoriala

  • czysty kontroler
  • czyszczenie config podczas wgrywania .bin

Cofnąłem się jedną wersje wstecz, to samo.

@krzaczek
Copy link

krzaczek commented Feb 18, 2020

Jeżeli chodzi o AQI to mam ten sam problem.

Pomaga zmiana tej lini

json["AQI_ECO_HOST"] = AQI_ECO_HOST;

na

json["AQI_ECO_HOST"] = String(AQI_ECO_HOST);

Analogicznie jak jest AQI_ECO_PATH

Ogólnie to nie jest problem z samym zapisem configa bo on się zapisuje. Problem jest taki że część pół jest zapisywana pusta np AQI_ECO_PATH ale też DUST_MODEL - widać to ładnie na debugu.

Postaram się wieczorem podesłać logi z konsoli.

@bfaliszek
Copy link
Member

bfaliszek commented Feb 18, 2020

@krzaczek
dodałem sugerowane przez Ciebie zmiany do kodu. Są na razie w branchu dev - https://github.com/hackerspace-silesia/Smogomierz/tree/dev
pełna lista zmian - 00bb837
W zasadzie to dopisałem wszędzie typy zmiennych.

@bfaliszek bfaliszek changed the title Nie zapisuje ustawień po wyborze czujnika zanieczyszczeń Problem z zapisem ustawień Feb 18, 2020
@bfaliszek bfaliszek added the bug label Feb 18, 2020
@bfaliszek
Copy link
Member

Sprawdźcie teraz, bo możliwe, że problem leżał gdzie indziej i przy zmianie miernika PM pobierały zły plik .bin z serwera.

@michlimek
Copy link

michlimek commented Apr 19, 2020

Też natrafiłem na ten problem z zapisem ustawień AQI_ECO. Przy używaniu gotowej paczki wszystko jest ok, jednak przy samodzielnej kompilacji kodu nie. Znalazłem w pliku spiffs.cpp podwójny zapis ustawień. Usunięcie jednego u mnie pomogło.

AQI_ECO_ON = json["AQI_ECO_ON"];
  _safeCpy(AQI_ECO_HOST, json["AQI_ECO_HOST"], "host");
  _safeCpy(AQI_ECO_PATH, json["AQI_ECO_PATH"], "path");``

@bfaliszek
Copy link
Member

bfaliszek commented Apr 20, 2020

@michlimek Poprawiłem w branchu Dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants