Skip to content

Update to nimble 1.3 master branch#198

Merged
JF002 merged 1 commit intodevelopfrom
update-nimble-1_3-master
Feb 4, 2021
Merged

Update to nimble 1.3 master branch#198
JF002 merged 1 commit intodevelopfrom
update-nimble-1_3-master

Conversation

@JF002
Copy link
Copy Markdown
Collaborator

@JF002 JF002 commented Feb 2, 2021

I created this PR to check if newer version of Nimble would improve BLE connection reliability.
Fow now, InfiniTime uses the last tagged version (1.3.0) of Nimble.
This branch updated nimble to the last commit in the master branch (not tagged).

I did many succesful OTA based on this branch. Anyone is free to test it and report success, failure, bugs or improvement :)

…c38b2b0dbcfe1

WARNING : heartbeat task is disabled!
@JF002
Copy link
Copy Markdown
Collaborator Author

JF002 commented Feb 2, 2021

Here is a DFU build of this branch (WARNING : this is a test FW, use it with caution on sealed devices) dfu-0.11.0.zip

@piggz
Copy link
Copy Markdown
Contributor

piggz commented Feb 2, 2021

If you have it connected to Amazfish and leave it for some time, does amazfish remain connected?

@petterhs
Copy link
Copy Markdown
Contributor

petterhs commented Feb 3, 2021

This is massive! Thanks! While I had 4 or 5 failed OTA's just to get to this branch, I have yet to get a disconnect while OTA updating from this branch. Somewhere around 10 straight successful OTA updates.

Pros:

  • The recovery/restart of the update after an error is working. (Though with a visual bug) Before when i got an error I absolutely had to reboot the watch but now it reconnects automatically and retries the update.

Cons:

  • Visual bug that persists across screens. Happens when OTA update starts while the watch is on the error screen, either automatically or manually.
  • Hard to debug above bug because it never disconnects...lol. I have to walk to the other side of my apartment.

@piggz
Copy link
Copy Markdown
Contributor

piggz commented Feb 3, 2021

After flashing with amazfish on my phone, I paired with my desktop and it has maintained a connection for over 30 minutes now.

@JF002
Copy link
Copy Markdown
Collaborator Author

JF002 commented Feb 3, 2021

My devkit stayed connected the whole day with my Android phone!

So, it looks like this upgrade improved the situation by a lot! I guess I'll clean the code, merge the branch and release a new version soon :)

@JF002
Copy link
Copy Markdown
Collaborator Author

JF002 commented Feb 3, 2021

I pushed an update to re-enable the heart rate task. Here is the dfu.
dfu-0.11.0-test.zip

@JF002 JF002 changed the title TEST : Update to nimble 1.3 master branch Update to nimble 1.3 master branch Feb 4, 2021
@JF002 JF002 merged commit 6259b81 into develop Feb 4, 2021
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

Successfully merging this pull request may close these issues.

3 participants