[IntroRemoteDS-pre] Execution Cell Count & Output not cleared in L3_DataPreparation #380
Labels
Type: Improvement 📈
Performance improvement not introducing a new feature or requiring a major refactor
Comments
leriomaggio
added
the
Type: Improvement 📈
leriomaggio
added a commit
to leriomaggio/courses
that referenced
this issue
Nov 17, 2021
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)
4 tasks
leriomaggio
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Cell output (and corresponding cell execution count) have not been cleared in
L3_DataPreparation.ipynbnotebook.Just noticed as I started working on Lesson 3 in the course Pre-release :)
Are you interested in working on this improvement yourself?
Additional Context
Also execution counts are not in consecutive order, so this might be even more confusing.
Happy to send a PR myself, but I think it would be quicker if you'd prefer doing it on your end.
HTH
The text was updated successfully, but these errors were encountered: