Skip to content

Fix #371 #372

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

Merged
merged 1 commit into from
Mar 14, 2016
Merged

Fix #371 #372

merged 1 commit into from
Mar 14, 2016

Conversation

BibMartin
Copy link
Contributor

Hi @ocefpaf,
this needs to be cherry-picked on the branch v0.2.
I don't know how we should deal with this: shall we do a v0.2.1 to incorporate the fix ? wait for more bugfixes and do a v0.2.1 in a few weeks ?

@BibMartin
Copy link
Contributor Author

I cannot figure out what's going wrong with the tests (and cannot reproduce it)

@ocefpaf
Copy link
Member

ocefpaf commented Feb 24, 2016

I cannot figure out what's going wrong with the tests (and cannot
reproduce it)

I am away in a conference. I'll take a look, merge, and backport to a
release as soon as I get back.

PS0: since this is a bugfix we should cut a v0.2.1, but no need to wait for
more stuff. We can always cut another v0.2.x series.

PS1: The failure is relate to conda and not folium. I am already looking
into it for another project.

@BibMartin
Copy link
Contributor Author

Ok, great! no hurry, we have a workaround in #371 meanwhile.

@BibMartin
Copy link
Contributor Author

@ocefpaf Oups, it seems we forgot this one.
If you have time to review, merge and cherry pick...

ocefpaf added a commit that referenced this pull request Mar 14, 2016
@ocefpaf ocefpaf merged commit 4e68c72 into python-visualization:master Mar 14, 2016
@ocefpaf
Copy link
Member

ocefpaf commented Mar 14, 2016

This was disappeared from both our radars 😉

sanga pushed a commit to sanga/folium that referenced this pull request Oct 24, 2017
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