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

Put old DIA=3 bilinear information behind a details tag #1551

Merged
merged 3 commits into from
Nov 13, 2020

Conversation

loztblaz
Copy link
Contributor

This isn't anywhere near a clean edit, but a lot of this information is years out of date and contradictory with other parts of the docs. Recreating all of the graphs with accurate to recent version DIAs is a lot of work, but constant disclaimers that graphs do not represent current information would bloat this section a lot. Considering that the vast majority of readers of this page would be on an exponential curve with a DIA of 5 or more, it seems confusing to have that outdated info front and center. Folding to the exponential curve info with a details tag seems better.

This isn't anywhere near a clean edit, but a lot of this information is years out of date and contradictory with other parts of the docs. Recreating all of the graphs with accurate to recent version DIAs is a lot of work, but constant disclaimers that graphs do not represent current information would bloat this section a lot. Considering that the vast majority of readers of this page would be on an exponential curve with a DIA of 5 or more, it seems confusing to have that outdated info front and center. Folding to the exponential curve info with a details tag seems better.
@danamlewis danamlewis merged commit 5bffe4e into openaps:master Nov 13, 2020
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