search

movestore / cargo-agent-r

Analyzing app output running in R
GNU General Public License v3.0
0 stars 2 forks source link

New IO-Type: CTMM UD with Data #20

Closed jmsigner closed 1 year ago

jmsigner commented 1 year ago

I added a new IO type: ctmm UD with data. This is a list with three entries: 1) a list of fitted ctmm models, 2) a list ctmm UDs and 3) a list of ctmm telemetry objects that were used to fit the models.

andreakoelzsch commented 1 year ago

noch eine Frage: es scheint mir etwas viel, hier alle 3 listen reinzutun (model, UD und data), da der IOtyp "CTMM model and data" dann ja ein subset von diesem Typ ist, oder? Braucht man das Model, oder reichen UD und data vielleicht auch? Ich kenne mich nicht so ganz aus, was im workflow weitergereicht werden muss und gehe davon aus, dass ich das schon diskutiert habt, nur nochmal als letzter check... ? Merci!

jmsigner commented 1 year ago

Hallo Andrea,

gute Frage, ich habe auch ein etwas hin und her überlegt und bin mir nicht ganz sicher, ob es nötig ist das Model noch mitzunehmen. Ich dachte dann aber, dass es relevant ist welches Model für z.B. aKDE verwendet wurde und hab mich dann dafür entschieden. Wir können das aber gerne nochmals überdenken.

On Thu, May 25, 2023 at 11:35 AM Andrea Kölzsch @.***> wrote:

noch eine Frage: es scheint mir etwas viel, hier alle 3 listen reinzutun (model, UD und data), da der IOtyp "CTMM model and data" dann ja ein subset von diesem Typ ist, oder? Braucht mal das Model, oder reichen UD und data vielleicht auch? Ich kenne mich nicht so ganz aus, was im workflow weitergereicht werden muss und gehe davon aus, dass ich das schon diskutiert habt, nur nochmal als letzter check... ? Merci!

— Reply to this email directly, view it on GitHub https://github.com/movestore/cargo-agent-r/pull/20#issuecomment-1562593872, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGGQA463UCL3EOTJ3ZTPCLXH4RXZANCNFSM6AAAAAAYOIUQXE . You are receiving this because you authored the thread.Message ID: @.***>

andreakoelzsch commented 1 year ago

Danke Johannes, lasst es ruhig mal so für jetzt. Ist ja erstmal nur auf ctmm.moveapps.org verfügbar. Behaltet es im Kopf und vielleicht nochmal mit Anne oder Chris drüber sprechen, auch mit der Idee, dass andere user noch Apps mit diesen IO types entwickeln können. Die Liste mit den Modellen kann man ja kurz vor Ende nochmal rauswerfen... Dieser neue IO typ sollte jetzt auf ctmm.moveapps.org integriert sein, sobald du den IO typ request stellst und ich den approve. Viele Grüße