Skip to content

Use current scheduled basal after the last temp basal ends - #799

Open
bjorkert wants to merge 1 commit into
devfrom
fix/scheduled-basal-after-last-temp
Open

bjorkert wants to merge 1 commit into
devfrom
fix/scheduled-basal-after-last-temp

Conversation

@bjorkert

@bjorkert bjorkert commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

When no temp basal is running, the Basal row shows the scheduled rate at the current time, and the basal graph from the end of the last temp basal follows the schedule in the profile time zone, stepping at each schedule entry, at midnight and across daylight saving changes.

Before and after, with the last temp basal ending at 08:50 and the schedule changing from 0.65 to 0.5 at 08:55:

mock-before mock-after

When no temp basal is running, the Basal info value is the scheduled rate at the current time, and the basal graph from the end of the last temp basal follows the schedule in the profile time zone, stepping at each schedule entry.
@oddst

oddst commented Oct 29, 2026

Copy link
Copy Markdown

A little wish is that a check is also done to verify if insulin suspend delivery is on or not...! Or that the pod might be physically out of insulin...! This is something that I believe that both Trio and Loop both suffer with...!

@bjorkert

Copy link
Copy Markdown
Member Author

A little wish is that a check is also done to verify if insulin suspend delivery is on or not...! Or that the pod might be physically out of insulin...! This is something that I believe that both Trio and Loop both suffer with...!

Thanks, I'm looking into this in a separate mr to avoid scope creep.

@dnzxy

dnzxy commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

A little wish is that a check is also done to verify if insulin suspend delivery is on or not...! Or that the pod might be physically out of insulin...! This is something that I believe that both Trio and Loop both suffer with...!

That goes entirely contrary to how pump
managers interact with pumps and report pump
status.

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