bootcamp_machine-learning icon indicating copy to clipboard operation
bootcamp_machine-learning copied to clipboard

ML02 ex05 (mylinearregression): multiple issues (examples, syntax)

Open mli42 opened this issue 3 years ago • 0 comments

  • Day: 07
  • Exercise: 05

There are 4 issues on this exercice:


1) Init of MyLR

In the example, there is:

mylr = MyLR([[1.], [1.], [1.], [1.], [1]])

Thus giving a list of list, whereas in the previous module, we are given a np array:

lr1 = MyLR(np.array([[2], [0.7]]))

  1. mylinearregression

In the previous module, the class was named my_linear_regression (with _), which, imo, is more readable, and it should have the same name for this module. It is currently not producing any bug, but it is a bit inconsistent.


  1. mylr.theta

In the examples, it tries to access mylr.theta, but in the previous module the variable was named thetas with an S. A bit inconsistent too.


  1. loss / loss elem

In order to satisfy the examples, I noticed the division by 2m is performed during loss() and not loss_elem(). Imo loss should be only the sum of loss_elem, as it was in the previous module.


Fixed on:

  • [ ] Github
  • [ ] Gitlab

mli42 avatar Sep 05 '22 12:09 mli42