Skip to content

Pretty print pump events#2403

Merged
marionbarker merged 5 commits intoLoopKit:devfrom
loopandlearn:pretty-print-pump-events
Feb 14, 2026
Merged

Pretty print pump events#2403
marionbarker merged 5 commits intoLoopKit:devfrom
loopandlearn:pretty-print-pump-events

Conversation

@marionbarker
Copy link
Contributor

@marionbarker marionbarker commented Feb 11, 2026

Purpose

This is in response to Issue #2383.

This PR along with LoopKit PR 586 modify the formatting for the pump event history so the detailed view is easier to read.

@marionbarker
Copy link
Contributor Author

Test

Success.

Summary: This modification makes the pump history feature much more readable.

Configuration

Configuration: start with LoopKit/LoopWorkspace branch update_dev_to_3.11.1, commit c615911

  • Modify the commit for both Loop and LoopKit to point to loopandlearn:pretty-print-pump-events branch.
  • Build and install on an SE 3rd gen phone running iOS 26.2.1
  • Attach an MDT 515 pump to the Loop app
  • Dosing Strategy: Automatic Bolus

(Note - examples in Loop Issue #2383 gave Omnipod results. Test this PR pair with MDT results.)

Results using a Medtronic pump

Bolus

Pump history events remain mutable until confirmed with the pump manager.

Graphics below show an isMutable: true automatic bolus (left) and an earlier finalized manual bolus (right).

loop-loopkit-pretty-print-bolus

Temp Basal

  • Modify Dosing Strategy to Temp Basal Only

  • Enter carbs with no bolus to trigger an automatic temp basal event.

Graphics below show an isMutable: true temp basal delivery (left) and after that temp basal is finalized (right).

loop-loopkit-pretty-print-tempbasal

@marionbarker marionbarker requested review from itsmojo and ps2 February 11, 2026 17:03
@marionbarker
Copy link
Contributor Author

Updated screen shots

Made a few modifications to both Loop and LoopKit.

  • Request via PM from @itsmojo to label the syncIdentifier.
  • I added the date to time labels because
    1. more than one date is encompassed in history
    2. screenshots provided by users asking for help will provide date, avoiding possible confusion
  • reordered some fields
    • assist in answering these very common questions online:
      • was this a manual dose or did Loop automatically provide this dose?
      • did the Temp Basal actually give me any insulin
    • put less useful lines near the bottom
      • the start and end time is useful but not looked at as often
      • The manuallyEntered and wasProgrammedByPumpUI are for pumps like Medtronic with pump user interface to dose independent of Loop
      • the description and scheduledBasalRate are (at the current time) always nil

Graphic shows before (left) and after (right) adding the date to the main history list:

add-date

Graphic shows before (left) and after (right) adding the date and reordering fields for detailed view

The initial detailed view for that bolus (left image) was acquired before the dose was finalized.

details-date-rearranage

@marionbarker
Copy link
Contributor Author

Added the syncIdentifier to the dose formatted string (LoopKit PR 586)
Remove the syncIdentifier from the string provided by Loop (this PR)

Only difference with respect to the graphic on the right side in the previous comment is no blank line between scheduledBasalRate and syncIdentifier.

IMG_7315

Copy link
Contributor

@itsmojo itsmojo left a comment

Choose a reason for hiding this comment

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

Looks like some good improvements. Thanks Marion!

@marionbarker
Copy link
Contributor Author

The date style (with commit 90e5b92) is not localized.

date-is-not-localized

Copy link
Collaborator

@ps2 ps2 left a comment

Choose a reason for hiding this comment

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

LGTM!

@marionbarker marionbarker merged commit c55fa49 into LoopKit:dev Feb 14, 2026
@marionbarker marionbarker deleted the pretty-print-pump-events branch February 14, 2026 21:59
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