Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

Add ML Stats #114

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add ML Stats #114

wants to merge 1 commit into from

Conversation

mariusdkm
Copy link
Member

Just a draft for now.

Kp wie aktiv ich die nächsten Wochen werde, wegen Klausurenphase und so...

@@ -14,6 +17,40 @@ def from_utc(utc_time: str) -> datetime.datetime:
).astimezone(timezone("Europe/Berlin")).replace(tzinfo=None)


def train_model(self, train_x, train_y, **model_parameters):
Copy link
Member Author

@mariusdkm mariusdkm Feb 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@McToel ist des ok, dass ich hier auch noch des train reingepackt habe?
Des passt mMn eingentlich ziehmlich gut da rein, so hat man viele Webseiten ML Sachen in einem File/Klasse

Eigentlich wollte ich des zu der train_models Methode unten schreiben und nicht zu dieser Hilfsmethode lol

return est


parameters = {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das hier könnte man überlegen, ob man des nicht in die Config reinpackt

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant