carpentries-incubator / high-dimensional-stats-r

High-dimensional statistics with R
https://carpentries-incubator.github.io/high-dimensional-stats-r
Other
12 stars 18 forks source link

Self-review notes #7

Closed alanocallaghan closed 2 years ago

alanocallaghan commented 3 years ago

Some notes for revisions after largely finishing content.

In general:

~- 04:~ moved to #65 ~- [ ] machine learning course isn't carpentries and should be linked (or an alternative~ ~- [ ] forward stepwise description a bit sparse~ ~- [ ] forward/reverse could do with more detail~

~- 09: Moved to #64~ ~- [ ] exercise 1 bugged~ ~- [ ] initial mcq before callout?~ ~- [ ] title for 1st practical bit~ ~- [ ] formal description of silhouette width~ ~- [ ] k of 5 or k=5, not both~ ~- [ ] title for introducing bootstrap~ ~- [ ] title for applying bootstrap~ ~- [ ] more detail on bootstrap~

~- 10:~ moved to #65 ~- [ ] explain densfun warnings (good Q)~ ~- [ ] error in dnorm, exercise 1~ ~- [ ] warnings in histograms (suppress?)~ ~- [x] #17~ ~- [ ] more detail on fitting a bivariate normal~

catavallejos commented 2 years ago

Trying to close old issues to I plan to transfer open things to #64 where Ailith compiled a main list of planed changes. I am updating this issue accordingly.

catavallejos commented 2 years ago

@Alanocallaghan in the list above, 03 and 04 are in the reverse order no?

alanocallaghan commented 2 years ago

That'll be the order they were in at the time, I believe

catavallejos commented 2 years ago

@Alanocallaghan for 03 you wrote: 2nd exercise a bit on-the nose ... unless you have specific comments, I will skip

alanocallaghan commented 2 years ago

I think that's referring to the fact that it's asking students exactly what they've just been told for q1, and q2 essentially recaps the normality of residuals assumption, which on reflection we shouldn't really emphasise given it's not super important. Can be ignored

catavallejos commented 2 years ago

Thanks @Alanocallaghan. I will ignore for now and close this issue as all outstanding items have been moved to our current 'master' to-do list in #64 or correspond to optional episodes (moved to #65)