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

J chan hkust filename patch #27

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

Conversation

j-chan-hkust
Copy link

Hey mate,

I just did a small edit to the code to allow for the option of saving files under a specified filename.

If filename is left blank, then there will be no behavior change, code should function exactly as before.

If you do specify a filename (i.e. "heatmap.png") it will trigger a clause that will save the plot as a file.

Intended purpose is to make using the code to update analytics pages and dashboards easier.

gives user option to supply a filename to save the output image.
now can enter a filename into cal_heatmap. If filename is empty, then we don't save file
@j-chan-hkust
Copy link
Author

Example of usage:
activity_heatmap = heatmap(dates, counts, title='GTD Activity', value_label=True, cmap="YlGn", filename="heatmap")

@j-chan-hkust
Copy link
Author

Not sure if this is merged with the main branch yet and reflected in pip - when I tried to set things back up on my own machine, I re-remembered that I made this change, and for the sake of easier environment maintenance, would be nice to have this merged in.

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