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

Is sample_n described earlier? Use instead of slice_sample #161

Closed
avahoffman opened this issue Sep 20, 2024 · 3 comments · Fixed by #169
Closed

Is sample_n described earlier? Use instead of slice_sample #161

avahoffman opened this issue Sep 20, 2024 · 3 comments · Fixed by #169

Comments

@avahoffman
Copy link
Contributor

          - [ ] Is `sample_n` described earlier? Use instead of `slice_sample`

Originally posted by @avahoffman in #148 (comment)

@carriewright11
Copy link
Contributor

carriewright11 commented Sep 30, 2024

slice_sample() is introduced in the subsetting lecture. sample_n() has been superseded in favor of slice_sample()

@carriewright11
Copy link
Contributor

The subsetting lecture is before the data classes lecture, where you wrote this comment, so it will have already been introduced.

@avahoffman
Copy link
Contributor Author

Ok Great. This was part of a code review for @ehumph -- whoever gets to it first, let's get this fixed :)

@avahoffman avahoffman linked a pull request Oct 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants