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

[IntroRemoteDS-pre] Fix and improvements to L3_DataPreparation notebook #383

Open
wants to merge 6 commits into
base: introduction-to-remote-data-science
Choose a base branch
from

Conversation

leriomaggio
Copy link

Description

This PR fixes a few minors in the L3_DataPreparation.ipynb notebook
after coding through the first block of Lecture 3 in Introduction
to Remote Data Science pre-release.

This commit addresses Issue #380 #381 #382
Moreover:

  • duplicates in raw_data should be applied inplace
  • plt.show() has been added whenever a plot is generated to avoid the repr of the Axis object
  • the plot_extrapolated_country function had a missing an import numpy as np
  • x = list(range(53)) has been updated as per drop in-place
  • .iloc is always used whenever accessing an entry in the transposed DataFrame. (Without iloc, the code doesn't work - pandas 1.3.3)

Checklist

yemikifouly and others added 4 commits November 15, 2021 23:37
This commit fixes a few minors in the L3_DataPreparation notebook
after coding through the first block of Lecture 3 in Introduction
to Remote Data Science pre-release.

This commit addresses Issue OpenMined#380 OpenMined#381 OpenMined#382
Moreover:
- duplicates in raw_data should be applied inplace
- plt.show() whenever a plot is generated to avoid
the repr of the Axis object
- the plot_extrapolated_country function had a missing import
of numpy as np
- x = list(range(53)) has been updated as per drop inplace
- iloc is used whenever accessing an entry in the DataFrame.
(Without iloc, the code doesn't work)
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@leriomaggio leriomaggio changed the title Fix and improvements to L33 DataPreparation notebook [IntroRemoteDS-pre] Fix and improvements to L3_DataPreparation notebook Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants