Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Little issues in day01 ex01 ex02 ex3 ex04 #41

Closed
6 tasks done
quarkgluant opened this issue Jan 21, 2020 · 1 comment
Closed
6 tasks done

Little issues in day01 ex01 ex02 ex3 ex04 #41

quarkgluant opened this issue Jan 21, 2020 · 1 comment
Assignees
Labels

Comments

@quarkgluant
Copy link

quarkgluant commented Jan 21, 2020

  • Day: 01
  • Exercise: 01, 02, 03, 04
  • ex01 issue in the cost_elem docstring

Description:
Calculates all the elements 0.5M(y_pred - y)^2 of the cost function.

à la place de (0.5/M)*(y_pred - y)^2

  • ex02

arguments manquants de fit_

def fit_(theta, X, Y):

à la place de def fit_(theta, X, Y, alpha, n_cycle):

  • ex03
    `arguments manquants de fit_

def fit_(theta, X, Y):

à la place de def fit_(theta, X, Y, alpha, n_cycle):

  • ex03
    les valeurs données dans les exemples theta1, predict_(theta1, X1), theta2, predict_(theta2, X2)

theta1
array([[2.0023..],[3.9991..]])
predict_(theta1, X1)
array([2.0023..], [6.002..], [10.0007..], [13.99988..], [17.9990..])

correspondent à self.theta -= (0.5*alpha/M)*correct
à la place de self.theta -= (alpha/M)*correct

  • ex04
    erreurs dans les 2 exemples avec mse_

print(linear_model1.mse_(Xpill, Yscore))
57.60304285714282

à la place de print(linear_model1.mse_(Y_model1, Yscore))
et pareil pour le model2

  • ex05
    même erreur qu'à l'ex04 dans les exemples avec mse_
@ppeigne ppeigne self-assigned this Jan 30, 2020
ppeigne added a commit that referenced this issue Feb 2, 2020
@ppeigne ppeigne assigned ppeigne and madvid and unassigned ppeigne Feb 3, 2020
madvid pushed a commit that referenced this issue Feb 3, 2020
…urquoi mse_(X,Ytrue) et non pas mse_(Ypred, Ytrue)
madvid pushed a commit that referenced this issue Feb 3, 2020
…des exemples, il y avait bien l'erreur mentionnee dans la methode fit (facteur de 0.5).
@madvid
Copy link
Collaborator

madvid commented Feb 3, 2020

Concernant l'erreur des exemples dans l'exercice 2 (et non pas exercice 3):
Il y avait effectivement une erreur (presence du facteur 0.5) dans la formule de la descente de gradient de la methode fit, les valeurs de theta1, predict(theta1,X1), theta2 et predict(theta2,X2) post fit sont desormais corrigées.

Concernant l'erreur de le 04 et 05:
Une précision a été apporté comme quoi il faut appeler predict au sein de mse (dans l'exercice 4), celle-ci spécifie aussi que les 2 arguments seront bien X et Ytrue comme arguments dans les methodes des metrics des exercices qui suivent et qu'il faut donc appeler predict au sein de la methode de la metric.

@madvid madvid closed this as completed Feb 3, 2020
llenotre pushed a commit that referenced this issue Feb 25, 2022
llenotre pushed a commit that referenced this issue Feb 25, 2022
…urquoi mse_(X,Ytrue) et non pas mse_(Ypred, Ytrue)
llenotre pushed a commit that referenced this issue Feb 25, 2022
…des exemples, il y avait bien l'erreur mentionnee dans la methode fit (facteur de 0.5).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants