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

Data Collection Edits #27

Closed
wants to merge 8 commits into from
Closed

Data Collection Edits #27

wants to merge 8 commits into from

Conversation

amandakube
Copy link
Collaborator

Made some changes to my chapter (Now ch 10, before ch 11)

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@amandakube amandakube closed this Nov 2, 2022
@amandakube amandakube deleted the chapter11 branch November 2, 2022 22:09
@amandakube amandakube restored the chapter11 branch November 2, 2022 22:13
@amandakube
Copy link
Collaborator Author

Closed by mistake sorry!

@amandakube amandakube reopened this Nov 2, 2022
@jesteria
Copy link
Member

jesteria commented Nov 3, 2022

FYI: I pushed some minor clean-up to the branch.

Copy link
Member

@jesteria jesteria left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can merge this; but, it looks like the Biases section – which was split into a separate section – is also still in the Sampling section (repeated).

@amandakube
Copy link
Collaborator Author

amandakube commented Nov 3, 2022 via email

@amandakube amandakube closed this Nov 3, 2022
@jesteria jesteria added the content textbook content label Nov 3, 2022
@jesteria
Copy link
Member

jesteria commented Nov 3, 2022

Cool, I think. Did you mean to close this?

SusannaLange pushed a commit to SusannaLange/textbook-datascience-1 that referenced this pull request Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content textbook content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants