Quarto mod8 mod9 #52
Quarto mod8 mod9 #52
Conversation
|
| # Plot your confusion matrix on your test set | ||
| ____ | ||
| # Leave the following line commented out for the checking purposes. Uncomment to see the plot. | ||
| #plt.show() |
There was a problem hiding this comment.
I copied over the solution to the exercise field but I am getting this error: "Error while checking multi_class_revised: "You need to plot a confusion matrix.""
There was a problem hiding this comment.
I have updated the error message to make it clearer that we need to comment out plt.show() for checking purposes.
| <iframe | ||
| class="video" | ||
| src="https://www.youtube.com/embed/JK9569FeWLg" | ||
| title="Module 8 Video - What Did We Just Learn?" |
There was a problem hiding this comment.
It seems the video was cut off a few seconds too early and stops mid-sentence at 'We look forward...'. Could we ask the team to adjust it?
| <iframe | ||
| class="video" | ||
| src="https://www.youtube.com/embed/ELIEplv266Y" | ||
| title="Module 9 Video - Congratulations!" |
There was a problem hiding this comment.
Also here the video seems to finish one min earlier than in the "original version". Is there a reason for that?
|
@sandygross I have addressed all comments except those related to the videos. Please check again. Thank you so much! |
joelostblom
left a comment
joelostblom
left a comment
There was a problem hiding this comment.
Great job @zmx721 ! Two minor changes requested and then we can merge.
| | ||
|
|
||
| <font size="4"><em> predicted(price) = coefficient<sub>bedrooms</sub> x #bedrooms + coefficient<sub>bathrooms</sub> x #bathrooms + coefficient<sub>sqfeet</sub> x #sqfeet + coefficient<sub>age</sub> x age + intercept </em></font> | ||
|
|
||
|
|
||
| <font size="4"><em> predicted(price) = 0.03 x #bedrooms + 0.04 x #bathrooms + 0.002 x #sqfeet + -0.01 x #age + intercept </em></font> | ||
|
|
||
|
|
||
| <font size="4"><em> predicted(price) = (0.03 x 3) + (0.04 x 2) + (0.002 x 1875) + (-0.01 x 66) + 0 </em></font> | ||
|
|
||
| <font size="4"><em> predicted(price) = 3.26 </em></font> |
There was a problem hiding this comment.
This currently flows together in a way that makes it hard to read:
Could you try either adding <br> tags to separate the lines or maybe even formatting as math with surrounding $?
There was a problem hiding this comment.
On my end, it appears as shown in the screenshot below, before any changes were made. Resizing is also fine.
I added a <br> tag. Does it help on your side?
modules/module8/module8-14-probabilities_and_logistic_regression.qmd
Outdated
Show resolved
Hide resolved
|
@joelostblom Please check if the issue is fixed. Thank you. |
joelostblom
left a comment
joelostblom
left a comment
There was a problem hiding this comment.
Great, this looks good now, thank again!
No description provided.