Hi,

I'm currently heavily relying on your module to plot calendar heatmap. Your module is great but I wonder If you know a simple way to put a colorbar for the whole figure on the left of the calendar heatmap figure. It could also be a neat feature in your next iteration on the module to have an option to put automatically the colorbar when creating the heatmap.

When run with Matplotlib 2.0.2 (Python 3.6.2) I get the following deprecation warnings:

/.../venv/lib/python3.6/site-packages/calmap/__init__.py:128: FutureWarning: how in .resample() is deprecated
the new syntax is .resample(...).sum()
  by_day = data.resample('D', how=how)
/.../venv/lib/python3.6/site-packages/calmap/__init__.py:146: MatplotlibDeprecationWarning: The get_axis_bgcolor function was deprecated in version 2.0. Use get_facecolor instead.
  linecolor = ax.get_axis_bgcolor()

Just a heads up for future proofing.

In version 1.0.0 of Pandas, Series.ix and DataFrame.ix was removed, breaking calmap. See https://pandas.pydata.org/docs/whatsnew/v1.0.0.html

However, this method is still used at https://github.com/martijnvermaat/calmap/blob/83e2a9a0bdc773c9e48e05772fb412ac8deb8bae/calmap/init.py#L220

I got the following warning and make this patch.
/usr/local/lib/python2.7/site-packages/calmap/init.py:295: FutureWarning: how in .resample() is deprecated the new syntax is .resample(...).sum()

by_day = data.resample('D', how=how)

data.resample('D', how=how) is deprecated and use data.resample('D').sum() instead.

Thanks for a useful calendar heatmap package. Is it easy to add the two above features? For the 1st one, an example is black lines in [https://bl.ocks.org/mbostock/4063318]

When I run the code on the doc page: import numpy as np; np.random.seed(sum(map(ord, 'calmap'))) import pandas as pd import calmap

all_days = pd.date_range('1/15/2014', periods=700, freq='D') days = np.random.choice(all_days, 500) events = pd.Series(np.random.randn(len(days)), index=days) calmap.yearplot(events, year=2015) I get the following result: Only when I change the aggregation to be mean, I get the expected result:

calmap.yearplot(events, year=2015,how=u'mean')

python version: 3.7.1 pandas version: 0.23.4 calmap version: 0.0.7 matplotlib version: 3.0.1

If fillcolor is specified, it's also used as the color for the yearlabels. This can be very impractical if you want days with no data to be shown in white. Then the yearlabels vanish.

To draw the lines, I had to modify some part of the code:

1. Added a "month" column to the dataframe.
2. If the first days of the year are part of last year's week (would be week 52), offset every week by 1 so they are now week 1, the previous week 1 is now week 2, etc. This is so that we know where to draw the lines.
3. I have added a small padding to the axes (0.3 of one cell) in order to avoid the outside lines being cut by the axes limit.
4. The month ticks position has been slightly pushed to the left so that their center now lines up with the center of the average week number.

This PR will break the tests because the output images are going to be different. This is not due to the month separator per se but to the slight esthetic changes (more margin and tick labels more centered). Would need to regenerate the baseline images.

For the fix, the two issues that were raised in the previous PR were:

1. The how argument to these functions is now ignored. Using the agg aggregation generic function, we can pass the how parameter.
2. It breaks compatibility with previous versions. Checking the version before and conditioning on having at least 0.18.0 solves this.

For the month separator, my solution was to draw a bunch of lines with a bit of logic to figure out how to deal with the first day of the month. I also slightly modified were the month labels were placed because they didn't line up very well.

when i use calmap.calendarplot without 'fillcolor', this is the result and when i use calmap.calendarplot with 'fillcolor', this is the result, it adds the label of year

Axes.get_axis_bgcolor() is deprecated, replaced with the new call.

https://matplotlib.org/api/_as_gen/matplotlib.axes.Axes.get_axis_bgcolor.html https://matplotlib.org/api/_as_gen/matplotlib.axes.Axes.get_facecolor.html

If I use your calmap function on my Linux Red Hat Enterprise distribution, I'll get the warning findfont: Font family ['Arial'] not found. Falling back to DejaVu Sans. Normally the warning will be just ignored, but if executed in the context of a cronjob it will cause the job to exit. In your code the Font is defined in Line 308:

ylabel_kws = dict(
        fontsize=32,
        color=kwargs.get('fillcolor', 'whitesmoke'),
        fontweight='bold',
        fontname='Arial',
        ha='center')

Could this be changed to fontfamily = 'serif' or to a parameter?

Since I don't have admin rights on this machine there is no way for me to just install the missing Font.