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

Bug fix for false +0 (or +0.0 mmol) bg deltas #545

Merged

Conversation

aug0211
Copy link
Contributor

@aug0211 aug0211 commented Jun 19, 2024

  • Address issue Erroneous +0 BG deltas reported #544
  • BUG FIX: xDrip incorrectly reported +0 (or 0.0 for mmol) BG deltas when delta was not actually 0 (or 0.0 for mmol) - this only occurred occasionally when BG deltas were around 1
  • This fixes this issue, tested and verified on live device for mgdl

- BUG FIX: xDrip incorrectly reported +0 (or 0.0 for mmol) BG when delta was not 0 (or 0.0 for mmol) - this only occurred occasionally when BG deltas were around 1
- This fixes this issue, tested and verified on live device for mgdl
@aug0211
Copy link
Contributor Author

aug0211 commented Jun 19, 2024

Not sure about the preferred way to submit a PR, I don’t think I have access to cut a new branch in the main fork. I can target this PR to another branch if desired.

@aug0211 aug0211 changed the title Big fix for false +0 (or +0.0 mmol) bg deltas Bug fix for false +0 (or +0.0 mmol) bg deltas Jun 20, 2024
@aug0211
Copy link
Contributor Author

aug0211 commented Jul 1, 2024

@JohanDegraeve @paulplant wanted to bump this one for your visibility.

@aug0211
Copy link
Contributor Author

aug0211 commented Sep 24, 2024

@paulplant bump

@paulplant
Copy link
Collaborator

Merging into the 5.4 staging branch. There is a small logic change that I'd like to make but want to check it makes sense first.

@paulplant paulplant changed the base branch from develop to staging-5.4 September 25, 2024 18:18
@paulplant paulplant merged commit 74248cd into JohanDegraeve:staging-5.4 Sep 25, 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.

2 participants