Skip to content

assignment_3 - #3

Open
vikrama33 wants to merge 1 commit into
mainfrom
assignment-3
Open

assignment_3#3
vikrama33 wants to merge 1 commit into
mainfrom
assignment-3

Conversation

@vikrama33

Copy link
Copy Markdown
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

I am adding new code to create clear visualizations, cleaning the dataset, and exporting summary data. I also made small changes to improve chart colors, labels, and layout for better readability.

What did you learn from the changes you have made?

I learned how small changes in code, colors, and labels can make charts easier to read and understand. I also learned the importance of cleaning data properly to get accurate and clear visual results.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I was also thinking about using Tableau to make interactive charts for easier data exploration.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

I faced issues with missing data and Excel file errors, but I fixed them by cleaning the data and installing the needed Python packages.

How were these changes tested?

I tested the changes by running the Python code and checking that the charts displayed the correct and clear results.

A reference to a related issue in your repository (if applicable)

This project was completed independently, so there are no related issues in a repository.

Checklist

  • I can confirm that my changes are working as intended

@vikrama33 vikrama33 changed the title I have updated the assignment assignment_3 Nov 7, 2025

@kunzhi-yu kunzhi-yu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Vikrama, nicely done. I would suggest making the text easier to read on the pie chart. Moreover, the y axis space for beer manufactures is way too small and it is hard to actually discriminate between the breweries. The color is also adds unnecessary complexity.

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