search

atecon / FEP

Forecast Evaluation Package for gretl
0 stars 0 forks source link

existierende getLoss-Funktion verstärkt einbauen #10

Open schreibe opened 1 year ago

schreibe commented 1 year ago

Wir haben schon getLoss(), momentan für linlin, linex, dlinex, quadquad. Dies sollten wir in verschiedenen anderen Funktionen einbauen, um damit mehr Auswahl bei der gewünschten Verlustfunktion zu haben. (Und dann perspektivisch noch andere Verlustfunktionen zu ermöglichen, aber das ist wieder ein anderes Sub-Thema.)

atecon commented 1 year ago

Hallo Sven, dieses Ticket bezieht sich doch sicherlich auf diesen Zweig, richtig? https://github.com/atecon/FEP/tree/moreLosses

Wie ist denn hier im Moment der Stand?

Ich könnte den aktuellen master Stand "hineinmergen" und dann kann hier dran weiterentwickelt bzw. der Zweig nach master überführt werden.

schreibe commented 1 year ago

Soweit ich mich erinnere, ist der Stand der, dass die modularisierte (getLoss o.ä.) Funktion an mind. einer Stelle eingebaut wurde. (Giacomini-White?) Und die Funkion wurde von mir glaube ich auch ein bisschen erweitert um manche Verlustfunktionen, die an anderer Stelle des Pakets schon vorkamen. D.h. theoretisch sollte es schon funktionieren, aber praktisch ist es ungetestet und es gibt bestimmt neue Bugs darin. Es ist wohl auch nicht systematisch zu Ende geführt, d.h. bei manchen Tests gibt es wahrscheinlich noch ad-hoc-Berechnungen der Verluste, statt durchgehend diese getLoss-Funktion zu benutzen. Ich habe nichts dagegen, dass master in diesen Zweig gemergt wird, aber würde an der Stelle wie gesagt Konflikte erwarten. Ich habe frühestens ab ca. 12.4. Zeit für diese Dinge.

atecon commented 1 year ago

Ich habe nichts dagegen, dass master in diesen Zweig gemergt wird, aber würde an der Stelle wie gesagt Konflikte erwarten. Ich habe frühestens ab ca. 12.4. Zeit für diese Dinge.

Ich habe nun den master-Stand reingemerged.

D.h. theoretisch sollte es schon funktionieren, aber praktisch ist es ungetestet und es gibt bestimmt neue Bugs darin.

Ja, das sample script läuft für den CG-Test nicht. Ich habe da 2-3 TODO und ein FIXME in den Code eingefügt. Diese Stellen müssten noch überdachte bzw. angepasst werden.