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

Misc enhancements & Bug fix - Progressbars, Rich tables, Coin and Dice probability spaces, Black formatting #90

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ksachdeva
Copy link

@ksachdeva ksachdeva commented Jan 15, 2022

This pull request contains multiple changes however those changes are in separate commits.

When merging you can either flatten all the changes or pull them all separately.

Here are the changes so far:
a) Bug fix when replacement=False
b) Black formatting (https://black.readthedocs.io/en/stable/)
c) Dice and Coin probability spaces.
d) Display of progress bar when simulating (https://rich.readthedocs.io/en/stable/progress.html)
e) Usage of Rich (https://rich.readthedocs.io/en/stable/tables.html) tables. This IMHO helps not only display more beautiful tables but also helps in not creating custom HTML etc

@ksachdeva ksachdeva changed the title Misc enhancements - Progressbars, Rich tables, Coin and Dice probability spaces, Black formatting Misc enhancements & Bug fix - Progressbars, Rich tables, Coin and Dice probability spaces, Black formatting Jan 15, 2022
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.

1 participant