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

Summary V2 API and API V2 cleanup #8029

Merged
merged 5 commits into from
Jun 14, 2023
Merged

Summary V2 API and API V2 cleanup #8029

merged 5 commits into from
Jun 14, 2023

Conversation

sulkaharo
Copy link
Member

@sulkaharo sulkaharo commented Jun 4, 2023

  • Creates new /api/v2/summary V2 API to load minimized current system state in one request for use in watchfaces and other display-only applications, optimised for small data load size and plotting
    ** Basal curve, preprocessed for plotting from temporary basals and profile
    ** Insulin and carb treatments
    ** Profile
    ** CGM data
    ** Use hours=n GET parameter to load data from most recent n hours, up to 48
  • Cleans V2 API implementation
  • Removes unnecessary server init calls in API calls that were unnecessary
  • Expose latest SBX state through the CTX object

…quest for use in watchfaces and other display-only applications

* Cleans V2 API implementation
* Removes unnecessary server init calls in API calls that were unnecessary
* Expose latest SBX state through the CTX object
@bewest
Copy link
Member

bewest commented Jun 4, 2023

Looks good; intent is refactor + new feature, no breaking changes, it looks like?

* Preprocess temp basals and profile to create a temp basal dataset
* Cleaner profile output
* Support hours GET parameter (up to 48 hours)
Further minize returned data
@sulkaharo sulkaharo changed the title EasyState V2 API and API V2 cleanup Summary V2 API and API V2 cleanup Jun 7, 2023
@bewest
Copy link
Member

bewest commented Jun 8, 2023

FWIW, I merged this with my nightscout-connect branch for easy of testing. #7983

@bewest bewest merged commit c2199c1 into dev Jun 14, 2023
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