search

zreptil / nightscout-reporter

a web app to create PDF documents from nightscout data
https://nightscout-reporter.zreptil.de/
BSD 3-Clause "New" or "Revised" License
59 stars 47 forks source link

Fix korrigiert den Issue #46 und funktioniert auch mit dem MultiInsulin-Feature #47

Closed gruoner closed 4 years ago

gruoner commented 4 years ago

war doch etwas leichter das Problem zu korrigieren - entweder du übernimmst es direkt oder trägst den Code von Hand in den Master

zreptil commented 4 years ago

Danke für den Pullrequest. Ich habe mir die Änderungen angeschaut und ihn mal eingebaut. Die IOB Kurven sehen damit bei mir etwas anders aus. Aber es könnte durchaus sein, dass sie so näher an der Realität sind. Ich schliesse das hier. Falls es da noch weitere Probleme gibt, einfach wieder ein neues Ticket aufmachen.

gruoner commented 4 years ago

Hallo Andi

warum packst du den maxInsulinEffectInMS direkt in die IOB Funktion? Ich hatte ihn absichtlich in die ReportData Klasse reingepackt, weil dort auch MultiInsulin und MultiCarbs ihre Repräsentation finden werden. Dann brauche ich hier nur beim Laden von MultiInsulin den maxInsulinEffectInMS neu zu setzen und alles andere funktioniert weiter wie bisher - das ganze Mergen wird viel einfacher weil's keine Konflikte provoziert.... Bist du bereit das noch anzupassen; dann warte ich mit dem Merge noch?