allow to change labelsize in plot

[angelaheumann]

Based on

https://stackoverflow.com/questions/40207784/how-to-edit-the-label-font-sizes-on-building-a-treemap-with-squarify-in-python/53590473#53590473

I would suggest allowing to pass the size of the labels.

[laserson]

Thanks for the contribution! One thing I am hesitant about is that you specially call out labelsize but there are potentially many properties that one would want to set. I see three alternatives that might be more general:

1. Pass the kwargs into ax.text as well. I am not certain, but I suspect this will not cause any problems with the ax.bar. Then you can pass whatever properties you want.
2. Take an explicit dictionary argument into the plot function that will be passed into the ax.text function call.
3. The plot function is somewhat of a convenience. If you look at the code, you'll see it's quite easy to generate your own plot. You could also consider just calling the MPL API directly to generate your plots.

[angelaheumann]

1. -> does not work because ax.bar does not know fontsize and throws the following error: Unknown property fontsize
2. -> should be the best way, but passing two dictionary arguments at the same time causes conflicts (kwargs bar & kwargs text).
3. Is fine for me, but changing the text is an important feature for plotting complex treemaps.

Any ideas?

[laserson]

My preference would be to go for option 2. Instead of the plot function taking **kwargs, you can have it take bar_kwargs=None and text_kwargs=None.

[angelaheumann]

-> Updated

Tested:

• no bar_kwargs, no test_kwargs passed There is still an exception which should be avoided: bar() argument after ** must be a mapping, not NoneType. There 2 variants: 1. Check if bar_kwargs and test_kwargs != None or 2. use {} as default. I decided to use 2.
• bar_kwargs={'alpha':.9}, text_kwargs={'fontsize':4}

[laserson]

Thanks for the contribution, @angelaheumann!!