Skip to content

Conversation

@Mukulikaa
Copy link
Contributor

This is the barebones of the air quality analysis tutorial. I have added some explanations so that people will have at least a high-level idea of the tutorial. I am going to add explanations and some more descriptive statistics soon.

Closes gh-110.

cc: @melissawm @rossbar

Copy link
Member

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

Thanks @Mukulikaa! A few initial comments:

  • Make sure you add your tutorial to the README and the applications.md file, so it's shown on the ToC for the rendered site.
  • I like that this is a simple application, but that means it might be better suited for beginners. In that case, we need to break down the code and explanations about what we're doing a little more. In particular, it might be nice to highlight the NumPy methods and functions being used in the computations.
  • This is a style issue, and you may very well disagree, but since we are using notebooks, I'd prefer if the comments were not made on the code, but as markdown text, whenever possible.
  • It would be nice to add a few links to references for the Wilcoxon test and the t-test, at least.

@Mukulikaa
Copy link
Contributor Author

Thanks, @melissawm! I'll include these in my next version.

@Mukulikaa Mukulikaa marked this pull request as ready for review January 11, 2022 14:09
@Mukulikaa Mukulikaa changed the title WIP: Analysis of air pollution levels before and after lockdown in Delhi, India Analysis of air pollution levels before and after lockdown in Delhi, India Jan 11, 2022
@Mukulikaa
Copy link
Contributor Author

The rendered tutorial is here. I have tried to include some explanations or links wherever suitable to make it more or else self-contained. Let me know if I should add more explanations or any other improvements!

Copy link
Member

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

Hi @Mukulikaa ! I have left a few tiny comments but overall this is really nice. I'm happy with it! Thanks 🎉

Co-authored-by: Melissa Weber Mendonça <[email protected]>
@Mukulikaa
Copy link
Contributor Author

Thanks, @melissawm!

@melissawm
Copy link
Member

I'll leave it up for a few days in case @rossbar wants to take a look - thanks @Mukulikaa !

Copy link
Collaborator

@rossbar rossbar left a comment

Choose a reason for hiding this comment

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

Thanks for this @Mukulikaa , very nice work!

I've left a few comments but there's no blockers - take them or leave them!

Copy link
Collaborator

@rossbar rossbar left a comment

Choose a reason for hiding this comment

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

Excellent work (as usual) @Mukulikaa, thanks!

@rossbar rossbar merged commit d1d1861 into numpy:main Jan 28, 2022
@Mukulikaa
Copy link
Contributor Author

Thanks, @rossbar and @melissawm, for all your help!

@melissawm
Copy link
Member

Congrats and thanks for all the hard work, @Mukulikaa !

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.

Proposal: Analysis of air pollution levels before and after lockdowns

3 participants