-
Notifications
You must be signed in to change notification settings - Fork 19
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
Updating setup.cfg for new pep changes #62
base: master
Are you sure you want to change the base?
Conversation
Added a horizontal bar chart (hbar)
Would be great if you could merge this. |
Hi, can you please point me to the pep that is offending? What is the error message? |
plotille/_graphs.py
Outdated
|
||
from ._colors import color | ||
from ._figure import Figure | ||
from ._input_formatter import InputFormatter | ||
from ._util import hist as compute_hist | ||
|
||
|
||
def hbar(counts, labels=Optional[List[str]], width=80, log_scale=False, linesep=os.linesep, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove typing information. I intentionally put no typing info in this project for now.
plotille/_graphs.py
Outdated
|
||
from ._colors import color | ||
from ._figure import Figure | ||
from ._input_formatter import InputFormatter | ||
from ._util import hist as compute_hist | ||
|
||
|
||
def hbar(counts, labels=Optional[List[str]], width=80, log_scale=False, linesep=os.linesep, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the difference between this and hist_aggregated? Is it removing the bins part and the addition of labels?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly. Just needed a horizontal bar chart for something.
Made the requested changes. Updated the setup.py with more info from your .toml. If that was redundant I can remove. |
New horizontal bar chart (hbar)