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

Update submodule OmniBLE and OmniKit #122

Merged
merged 3 commits into from
Apr 25, 2024
Merged

Conversation

bjornoleh
Copy link
Contributor

  • Prevent hang following restart if Pod Setup was interrupted

git submodule:
a92e9752994e7b143cdb007d3c7bcba0c0cc9214 CGMBLEKit (v3.1-166-ga92e975)
eba7d98b9a7995f50209f14f20cc1ac779b330ea G7SensorKit (heads/main)
c01eba63e94e9f6f2841a8835680c4e39c61b18d LibreTransmitter (heads/main)
2f535b3ca46825e82e0dd1b5ef9daccd53a3f0ca LoopKit (v3.0-1039-g2f535b3)
aa82a184e8243d30dc3b488ae17fd31602a3bd80 MinimedKit (heads/main-5-gaa82a18)
e15b18796a6ac0a08851b4ca2924cec669d669c5 OmniBLE (heads/dev)
30428b78809bf816b12b01ca2a05eb843ab46c99 OmniKit (heads/main)
a0e419da314d0ad42b41ccb04af73cd1fbf41257 RileyLinkKit (heads/dev)
a2ccad72a55600c28549ab86ab1964c0d6558868 TidepoolService (heads/dev_rc)
21d8657d727f26df76342188e580022bf4884714 dexcom-share-client-swift (v0.4.1-156-g21d8657)

- Prevent hang following restart if Pod Setup was interrupted

git submodule:
 a92e9752994e7b143cdb007d3c7bcba0c0cc9214 CGMBLEKit (v3.1-166-ga92e975)
 eba7d98b9a7995f50209f14f20cc1ac779b330ea G7SensorKit (heads/main)
 c01eba63e94e9f6f2841a8835680c4e39c61b18d LibreTransmitter (heads/main)
 2f535b3ca46825e82e0dd1b5ef9daccd53a3f0ca LoopKit (v3.0-1039-g2f535b3)
 aa82a184e8243d30dc3b488ae17fd31602a3bd80 MinimedKit (heads/main-5-gaa82a18)
 e15b18796a6ac0a08851b4ca2924cec669d669c5 OmniBLE (heads/dev)
 30428b78809bf816b12b01ca2a05eb843ab46c99 OmniKit (heads/main)
 a0e419da314d0ad42b41ccb04af73cd1fbf41257 RileyLinkKit (heads/dev)
 a2ccad72a55600c28549ab86ab1964c0d6558868 TidepoolService (heads/dev_rc)
 21d8657d727f26df76342188e580022bf4884714 dexcom-share-client-swift (v0.4.1-156-g21d8657)

Co-authored-by: Marion Barker <marionbarker@earthlink.net>
@marionbarker
Copy link
Contributor

marionbarker commented Apr 23, 2024

Can you hold off a day or two. There are 3 PR (2 for OmniBLE and 1 for OmniKit) that I plan to merge in the next day or so. The SHA will be updated when these are merged and will be in dev and main respectively. The odds of the missing commit for OmniBLE hitting anyone in real life is microscopic (fixes hang when app restarts between 0x03 and 0x08 messages). I only saw it a couple of times in dozens of tests where I was deliberately quitting out of Loop when trying to set up a new pod using the DASH rPi simulator. I had to set a breakpoint in the code for us to be able to trigger it when I tried to reproduce the problem so we could fix it.

These are the next 3 PR I plan to merge - probably tomorrow:

@marionbarker
Copy link
Contributor

And those 2 appName commits only affect the Uncertain Recovery screen that is not used by Open-iAPS anyway. So won't affect the translation strings.

bjornoleh and others added 2 commits April 25, 2024 08:18
…LoopKit/main

* OmniBLE e15b187...91abd9a (2):
  > Merge pull request 121 from LoopKit/improved_error_messages

* OmniKit 30428b7...f51fe35 (1):
  > Merge pull request 34 from LoopKit/enable_appName

git submodule
 a92e9752994e7b143cdb007d3c7bcba0c0cc9214 CGMBLEKit (v3.1-166-ga92e975)
 eba7d98b9a7995f50209f14f20cc1ac779b330ea G7SensorKit (heads/main)
 c01eba63e94e9f6f2841a8835680c4e39c61b18d LibreTransmitter (heads/main)
 2f535b3ca46825e82e0dd1b5ef9daccd53a3f0ca LoopKit (v3.0-1039-g2f535b3)
 aa82a184e8243d30dc3b488ae17fd31602a3bd80 MinimedKit (heads/main-5-gaa82a18)
+91abd9aba338903cc7bccd3a4c9df5dc4452cc1f OmniBLE (heads/dev)
+f51fe354ea6739ee09de922ed836d6844545b610 OmniKit (heads/main)
 a0e419da314d0ad42b41ccb04af73cd1fbf41257 RileyLinkKit (heads/dev)
 a2ccad72a55600c28549ab86ab1964c0d6558868 TidepoolService (heads/dev_rc)
 21d8657d727f26df76342188e580022bf4884714 dexcom-share-client-swift (v0.4.1-156-g21d8657)

Co-authored-by: Marion Barker <marionbarker@earthlink.net>
@bjornoleh bjornoleh changed the title Update submodule OmniBLE to e15b187 Update submodule OmniBLE and OmniKit Apr 25, 2024
@marionbarker
Copy link
Contributor

I confirm this brings in the latest OmniBLE and OmniKit versions.

@bjornoleh bjornoleh merged commit a62b597 into alpha Apr 25, 2024
This was referenced Apr 28, 2024
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.

None yet

2 participants