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

Trio predictions #304

Merged
merged 8 commits into from
Jun 18, 2024
Merged

Trio predictions #304

merged 8 commits into from
Jun 18, 2024

Conversation

bjorkert
Copy link
Contributor

Adds all 4 predictions from Trio
Code refactoring
Fix for transparent tabbar in settings view. Thanks to @petwall for this suggestion.

@bjorkert bjorkert requested a review from marionbarker June 15, 2024 17:12
@marionbarker
Copy link
Collaborator

The transition between a variety of Trio sites works perfectly in updating the prediction plot.
The transition from Trio to Loop updates all except the prediction.

  • I can force it to update the prediction by dragging down on the Glucose value
  • Wait for the next CGM update, the prediction line updates at that point

Copy link
Collaborator

@marionbarker marionbarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested with latest changes. Works as expected.

@marionbarker marionbarker merged commit 57e6350 into dev Jun 18, 2024
@marionbarker marionbarker deleted the openaps-predictions branch June 19, 2024 14:49
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