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

use newer arduino-esp32 #408

Closed
ThomasWaldmann opened this issue Mar 27, 2021 · 21 comments
Closed

use newer arduino-esp32 #408

ThomasWaldmann opened this issue Mar 27, 2021 · 21 comments
Milestone

Comments

@ThomasWaldmann
Copy link
Contributor

ThomasWaldmann commented Mar 27, 2021

short after the 1.0.5 release (~3w ago), there is now a 1.0.6 release based on IDF release/v3.3.5 85c43024c.

https://github.com/espressif/arduino-esp32/releases/tag/1.0.6

if there is no regression, we should do next release based on this.

@ThomasWaldmann ThomasWaldmann changed the title try arduino-esp32 1.0.6 use arduino-esp32 1.0.6 Mar 27, 2021
@ThomasWaldmann ThomasWaldmann added this to the V1.15.1 milestone Mar 27, 2021
@ThomasWaldmann
Copy link
Contributor Author

@rexfue please update with any insights after trying it out.

@ThomasWaldmann
Copy link
Contributor Author

upgraded to 1.0.6 and flashed again:

  • boards.txt needed patching again (see misc/arduino-esp32/... in our repo) - the patch still applies cleanly
  • after that, flashing from arduino ide worked again and multigeiger is ticking normally
  • did not do http data uploading tests yet...

@rexfue
Copy link
Collaborator

rexfue commented Mar 27, 2021

Don't know what happened, I can't get a stable WLAN connection with the normal Heltec Kit. Only the Wireless Stick connects easily and stable. I need to investigate further - but I cannot continue until later next week.

@M-a-r-i-a-n-n-e
Copy link
Contributor

@rexfue Same here. Just to be sure, I played around with delay and yield and threw the Lora-related Code out and built it with the original board environment, but it did not help. I also tried it with VSCode as well as with the Arduino IDE.
@ThomasWaldmann why should I patch the boards.txt? As far as I know I just have to configure it again and that's it?

I also tried other versions of espressif32 and they did not work either. Can someone provide an old binary that really worked on the Heltec WiFi Kit 32 v2? (The binary @WwwBoehriDe flashed did not work very well for me and all newer versions I tried in the previous weeks lead to WiFi connection losses more or less frequently. My settings always were: no sound, local password set, WiFi set, upload to both destinations.)

@ThomasWaldmann
Copy link
Contributor Author

ThomasWaldmann commented Mar 28, 2021

@M-a-r-i-a-n-n-e the patching is needed so one can set the partitioning from the arduino IDE's menu, see the .txt in the same directory as the patch (misc/arduino-esp32/...). It's required for the generic release-binary building.

All the binaries I released are available from github releases page.

@M-a-r-i-a-n-n-e
Copy link
Contributor

@ThomasWaldmann Which of your binaries is 100% sure to work with WiFi on the WiFi Kit?

Regarding the patch: unfortunately, it changes nothing for me. From you manual, I cannot select:

Flash size: “4MB(32Mb)”

@ThomasWaldmann
Copy link
Contributor Author

@M-a-r-i-a-n-n-e AFAIK, selecting the flash size is not needed as it is autodetected. At least that worked for my 4MB device.

For more infos, please read: https://github.com/ecocurious2/MultiGeiger/blob/master/misc/arduino-esp32/partition-menu-1.0.5.txt

@M-a-r-i-a-n-n-e
Copy link
Contributor

ok, well, then I have no idea, because your binaries from github also do not lead to a stable WiFi connection (tried it down to 1.13.0 already).

@ThomasWaldmann ThomasWaldmann modified the milestones: V1.15.1, V1.16.0 Apr 9, 2021
@ThomasWaldmann
Copy link
Contributor Author

From last meeting: people have issues with 1.0.6 and wifi, so let's continue with 1.0.5 for now.

@ThomasWaldmann ThomasWaldmann removed this from the V1.16.0 milestone Apr 9, 2021
@ThomasWaldmann ThomasWaldmann changed the title use arduino-esp32 1.0.6 use newer arduino-esp32 Apr 9, 2021
@ThomasWaldmann ThomasWaldmann added the later stuff not to be done right now, but better later label May 8, 2021
@ThomasWaldmann
Copy link
Contributor Author

So, 1.15.0 and 1.15.1 releases were / will be made with arduino-esp32 1.0.5.

Later releases should either stay on 1.0.5 or use something > 1.0.6.

Seems like arduino-esp32 devs are working on a 2.0 release (only alphas yet).

@M-a-r-i-a-n-n-e
Copy link
Contributor

M-a-r-i-a-n-n-e commented May 15, 2021

I tried it again with the current code and WiFi works well with my ESP32. Thank you for fixing!

GEIGER: Sending to Madavi ...
GEIGER: Sent to Madavi, status: ok, http: 200 200
GEIGER: Sending to sensor.community ...
GEIGER: Sent to sensor.community, status: ok, http: 201 201

@ThomasWaldmann
Copy link
Contributor Author

You mean 1.15.1 release?

@M-a-r-i-a-n-n-e
Copy link
Contributor

M-a-r-i-a-n-n-e commented May 15, 2021

Just the master branch with

PLATFORM: Espressif 32 (3.2.0) > Heltec Wireless Stick
HARDWARE: ESP32 240MHz, 320KB RAM, 8MB Flash
...
PACKAGES:

  • framework-arduinoespressif32 3.10006.210326 (1.0.6)
  • tool-esptoolpy 1.30000.201119 (3.0.0)
  • toolchain-xtensa32 2.50200.97 (5.2.0)
    ...
    Dependency Graph
    |-- U8g2 2.28.8
    | |-- SPI 1.0
    | |-- Wire 1.0.1
    |-- Adafruit BME680 Library 2.0.0
    | |-- Adafruit BusIO 1.7.3
    | | |-- Wire 1.0.1
    | | |-- SPI 1.0
    | |-- Wire 1.0.1
    | |-- SPI 1.0
    | |-- Adafruit Unified Sensor 1.1.4
    |-- Adafruit BME280 Library 2.1.3
    | |-- Adafruit Unified Sensor 1.1.4
    | |-- SPI 1.0
    | |-- Wire 1.0.1
    |-- Adafruit Unified Sensor 1.1.4
    |-- IotWebConf 3.0.2
    | |-- DNSServer 1.1.0
    | | |-- WiFi 1.0
    | |-- WebServer 1.0
    | | |-- WiFi 1.0
    | | |-- FS 1.0
    | |-- WiFi 1.0
    | |-- EEPROM 1.0.3
    | |-- ESPmDNS 1.0
    | | |-- WiFi 1.0
    | |-- Update 1.0
    |-- MCCI LoRaWAN LMIC library 3.3.0
    | |-- SPI 1.0
    |-- NimBLE-Arduino 1.2.0
    |-- SPI 1.0
    |-- HTTPClient 1.2
    | |-- WiFi 1.0
    | |-- WiFiClientSecure 1.0
    | | |-- WiFi 1.0
    |-- WiFiClientSecure 1.0
    | |-- WiFi 1.0

@M-a-r-i-a-n-n-e
Copy link
Contributor

@ThomasWaldmann Out of curiosity I also tested your 1.15.1 release binary. It still fails to connect to WiFi at all, so I do not understand why framework-arduinoespressif32 <3.2.0 (1.0.5) should be better than 3.2.0 (1.0.6). Compiling with previous versions (3.1.1, 3.1.0, 3.0.0, ...) did not work for me, as WiFi was unstable for my two WiFi Kit 32.

Compiling with framework-arduinoespressif32 3.2.0 (1.0.6) in VScode works for me, so I can give my Multigeigers another try.

@ThomasWaldmann
Copy link
Contributor Author

@M-a-r-i-a-n-n-e 1.0.5 being better than 1.0.6 is what i got from the first few posts in this ticket (including one from you) and also from feedback in a jitsi meeting.

@t-pi
Copy link
Contributor

t-pi commented May 16, 2021

Apparently the Arduino BLE stack also did his part in thwarting the stability... Since the move to NimBLE lib, both my MultiGeigers run rock-stable (no reboots anymore, the normally-on is currently at BLE packet counter 18424, i.e. ~51h run time and by my guess it has rolled over once already...)

@M-a-r-i-a-n-n-e
Copy link
Contributor

@t-pi In March, I even removed all BLE references from the code and as I wrote above, I could compile no version with stable WiFi at all and even the older version that @WwwBoehriDe initially flashed for me had issues with WiFi. So I guess that mainly the new framework-arduinoespressif32 did the trick for me. I'm really glad it works!

@ThomasWaldmann
Copy link
Contributor Author

OK, so I guess I'll make a new release soon, with:

  • arduino-esp32 1.0.6 release based on IDF release/v3.3.5 85c43024c.
  • nimBLE and IotWebConf update from master

I also linked a new patch from the crash-reboot ticket #314, could try to get that in also.

Does that sound good to everybody concerning WiFi stability?

@ThomasWaldmann
Copy link
Contributor Author

espressif/arduino-esp32#4980 (comment) < notable.

@ThomasWaldmann
Copy link
Contributor Author

https://ecocurious.slack.com/archives/CGBLH04GJ/p1622141846004600 < t.pi: 1.0.6 + nimBLE + local alarm PR uptime >= 7d.

@ThomasWaldmann ThomasWaldmann removed the later stuff not to be done right now, but better later label Jun 18, 2021
@ThomasWaldmann ThomasWaldmann added this to the V1.16.0 milestone Jun 18, 2021
@ThomasWaldmann
Copy link
Contributor Author

Building release 1.16.0 with arduino-esp32 1.0.6 now...

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

No branches or pull requests

4 participants