Skip to content

Add code to look at the head from true_copy_surveys_df#464

Merged
tobyhodges merged 2 commits intodatacarpentry:gh-pagesfrom
ricdea:03-true-copy
Apr 4, 2023
Merged

Add code to look at the head from true_copy_surveys_df#464
tobyhodges merged 2 commits intodatacarpentry:gh-pagesfrom
ricdea:03-true-copy

Conversation

@ricdea
Copy link
Copy Markdown
Contributor

@ricdea ricdea commented Jun 16, 2020

Added a few lines to look at the head from the copied data frame.

This ensures students see that the copy has not changed.

Otherwise the object is created but not used.

Copy link
Copy Markdown
Contributor

@maxim-belkin maxim-belkin left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution, Richard! 🏅
All changes look very reasonable. Only one request to break down the long line.

Comment thread _episodes/03-index-slice-subset.md Outdated
refer to the same exact DataFrame object. If either one changes the object,
the other will see the same changes to the reference object.

However - `true_copy_surveys_df` was created via the `copy()` function. It retains the original values for the first three rows.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you please break this line so that it's less than 100 characters long? I suggest breaking after the period.

@maxim-belkin maxim-belkin added the type:enhancement Propose enhancement to the lesson label Jun 16, 2020
@tobyhodges
Copy link
Copy Markdown
Member

I'm helping the current lesson Maintainers process outstanding pull requests on this repository, in preparation for transition to the new lesson infrastructure.

Thank you for contributing, @ricdea. I have gone ahead and broken up the long line as requested by Maxim above, and will now merge this PR.

@tobyhodges tobyhodges merged commit aed9a3d into datacarpentry:gh-pages Apr 4, 2023
zkamvar pushed a commit that referenced this pull request May 8, 2023
Add code to look at the head from true_copy_surveys_df
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:enhancement Propose enhancement to the lesson

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants