Quizz answer and explanations are out of sync

Hi !

I had again the ‘issue with the comma’ on

Comma

Here and later, we use the name data and target to be explicit. In scikit-learn, documentation data is commonly named X and target is commonly called y.

I know it should be fixed, and you told me how to show the right version following:

https://mooc-forums.inria.fr/moocsl/t/how-to-go-back-to-original-version-of-a-notebook/381/2

But now there is no more comma at all.

Here and later, we use the name data and target to be explicit. In scikit-learn documentation data is commonly named X and target is commonly called y.

Quizz

Quiz 8

when making mistakes on question 1, it says solution a and c (which is right), but the details explanations
says a and c are incorrect:

solution: a) c)

The HistGradientBosstingClassifier documentation is available here.

a) is incorrect: C is not a parameter of HistGradientBosstingClassifier.

c) is incorrect: verbose is a parameter of HistGradientBosstingClassifier which allows to set how much information is printed during the fit. Because verbose does not affect how the model learns it is not a hyperparameter.

e) is incorrect: coef_ is an attribute of HistGradientBosstingClassifier (the final _ indicates that it is estimated from data i.e. available after the fit) not a parameter.

But now there is no more comma at all.

Well spotted I think you are on par for winning the most thorough beta-tester award :wink:. Not only do you make good comments but then you come back to check that this has been done as you wanted it.

I think what happened in this case is that I already fixed it when I saw your comment and thought “good enough” but the additional comma does indeed make the sentence slightly easier to parse … so I fixed it for good this time (I promise but please double-check in any case).

The quiz thing is not expected and should be fixed indeed, probably a misalignment somewhere …

OK so I fixed the problem in the gitlab but no this needs to be fixed in FUN

It’s fixed in FUN also

1 Like