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

Feature more insulin reportings #64

Closed gruoner closed 2 years ago

gruoner commented 2 years ago

Hier das Feature für die multiplen Insuline aus xDrip und NS. image image image

zreptil commented 2 years ago

Hi Torsten,

vielen Dank für den Pull-Request. Ich analysiere es noch, aber es schaut ganz gut aus. 👍 Den Pull-Request werde ich nicht durchführen, aber wenn das alles so funktioniert, wie es bisher aussieht, dann werde ich das in den Master integrieren und Dich natürlich entsprechend im Commit erwähnen.

zreptil commented 2 years ago

@gruoner Nachdem ich das jetzt bei mir reingebaut habe und unmittelbar auf Fehler bei der Reporterstellung gestossen bin, weil das nicht sauber implementiert wurde, brauche ich mal eine Datenbasis dazu, um sehen zu können, wo der Vorteil davon sein könnte. Kannst Du mir einen Link zu Deinen Daten zukommen lassen? Dann schaue ich mir mal an, was bei den Reports rauskommt, wenn die Daten, die Dein Code da abfragen will, auch tatsächlich vorhanden sind.

gruoner commented 2 years ago

@gruoner Nachdem ich das jetzt bei mir reingebaut habe und unmittelbar auf Fehler bei der Reporterstellung gestossen bin, weil das nicht sauber implementiert wurde, brauche ich mal eine Datenbasis dazu, um sehen zu können, wo der Vorteil davon sein könnte. Kannst Du mir einen Link zu Deinen Daten zukommen lassen? Dann schaue ich mir mal an, was bei den Reports rauskommt, wenn die Daten, die Dein Code da abfragen will, auch tatsächlich vorhanden sind.

da kommst du so ohne weiteres nicht ran - das liegt alles im Intranet bei mir.... ich vermute aber ganz frech, dass es dir an der Insulin-Tabelle im nightscout server fehlen dürfte. Ich häng sie grad in den PR.

gruoner commented 2 years ago

aber du hast damit nicht ganz Unrecht..... damit das Ding funktioniert, braucht man die Insulin-Extension im Nightscout Server und die dürften nur extrem wenige (um nicht zu sagen einer) haben. Ich fürchte, dass braucht einen grossen IF - oder du lässt es mit dem PR und den Multiplen Insulinen (ich hab keine Ahnung, wer die im xDrip benutzt und gleichzeitig den NSR verwendet). Bei Dir ist noch niemand deswegen vorstellig geworden; oder?