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

Changes #5

Merged
merged 2 commits into from
Aug 23, 2023
Merged

Changes #5

merged 2 commits into from
Aug 23, 2023

Conversation

Chirag529
Copy link
Contributor

Hello @veeralakrishna,
Sorry for directly mentioning you like this, but I have updated the "A Visual History of Nobel Prize Winners" project. Here are the changes that I made.

  1. README File: Added a readme file, outlining an in-depth portrayal of project, it's key findings and conclusion.
  2. Requirements File: Added a requirements.txt file containing all the essential pre-requisites for running the program successfully.
  3. Data Cleaning and Pre-processing: I've added some Data Cleaning and Pre-processing logic to ensure that datasets are accurate, consistent, and ready for further analysis.
  4. Enhancement in Question 2: Resolved a problem in Question 2 where only the top ten countries were displayed, ignoring other important statistics such as prize share and winner's gender. To ensure thorough and accurate visualization of the necessary data, the problem was corrected.
  5. Resolution of Question 3, Part 1: Addressed an issue in Question 3, Part 1, where the displayed table was not rendering as expected. This has now been resolved.
  6. Question 6 clarification: Improved the code to retrieve and show information on the first woman to receive the Nobel Prize.

If you believe that these modifications will be beneficial, please accept this PR request.
Thank you very much.

@veeralakrishna veeralakrishna merged commit 8f53d66 into veeralakrishna:master Aug 23, 2023
1 of 2 checks passed
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