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

fix bar_label_kwargs so rotation works #240

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amnona
Copy link
Collaborator

@amnona amnona commented Oct 22, 2020

Add option for "rotation" parameter in barx_label_kwargs / bary_label_kwargs
(previous implementation it was not possible to change rotation since provided in the function explicitly)

@amnona
Copy link
Collaborator Author

amnona commented Nov 5, 2020

@RNAer merge?

@RNAer
Copy link
Collaborator

RNAer commented Nov 5, 2020

xticklabel_kwargs=None, yticklabel_kwargs=None, parameters can already do that

@amnona
Copy link
Collaborator Author

amnona commented Nov 8, 2020

How can you rotate the xticklabels? it had rotation hard coded so always 0 if xticklabels. and if you try to supply it in the xticklabel_kwargs, you get parameter defined twice...

@RNAer
Copy link
Collaborator

RNAer commented Nov 9, 2020

it's not hard coded - it's just a default and will be updated when you provide it in xticklabel_kwargs.

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