-
Notifications
You must be signed in to change notification settings - Fork 129
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
The contour_label_fontsize parameter in _pdp_contour_plot() causes TypeError #40
Comments
Change contour_label_fontsize parameter in _pdp_contour_plot() to fontsize This fixes Issue SauceCat#40
Change contour_label_fontsize parameter in _pdp_contour_plot() to fontsize This fixes Issue #40
Hi @SauceCat, I was wondering if there were plans coming up to push a new release version out to PyPI that included this fix. Thanks! |
This worked. |
Hi Wolemercy, Could you please explain exactly how you made it work? |
Hi Wole, I have resolved this. Thank you |
Cheers mate |
Hi, |
Aye aye
Save and close the file. You should be good to go. |
Hi, |
Oups, |
This helped me, especially @Wolemercy's comment! Thanks, guys. |
You're welcome!
…On Mon, 8 Jun 2020 at 10:48, johann ***@***.***> wrote:
Oups,
it works! (I forgoted to restart… once again).
Thanks a lot!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#40 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMQ4273NH6TQCIBTZ4PR3K3RVSXXBANCNFSM4F6PRFLA>
.
--
Best Regards,
'Wole Ajewole.
"We are what we repeatedly do. Excellence, then, is not an act but a habit"'
- Aristotle
|
Cheers mate!
…On Fri, 13 Nov 2020 at 07:46, JRBOH ***@***.***> wrote:
This helped me, especially @Wolemercy <https://github.com/Wolemercy>'s
comment! Thanks, guys.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMQ4273XBMQUFOFDMZCAMIDSPTI4HANCNFSM4F6PRFLA>
.
--
Best Regards,
'Wole Ajewole.
"We are what we repeatedly do. Excellence, then, is not an act but a habit"'
- Aristotle
|
Still waiting for the PyPI update ... |
Why it's not updated in the PyPI yet?? |
Hi, can you try installing from GitHub repo instead? Because this repo is
not actively maintaining currently.
…On Tue, 9 Mar 2021 at 4:17 PM, Veerala Hari Krishna < ***@***.***> wrote:
Why it's not updated in the PyPI yet??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADXNPFGI7MWB7H62GZ7JKRDTCXKTBANCNFSM4F6PRFLA>
.
|
Both installing from the repo and editing the file as @Wolemercy suggested work fine. However, it is really inconvenient to have to manually install PDPBox because of this bug instead of using pip and a requirements file. Since the repo is no longer supported I/we would be happy to at least have the latest version in PyPI. |
Understand, let me try to figure it out these days.
…On Tue, 9 Mar 2021 at 6:56 PM, Evgeni ***@***.***> wrote:
Both installing from the repo and editing the file as @Wolemercy
<https://github.com/Wolemercy> suggested work fine. However, it is really
inconvenient to have to manually install PDPBox because of this bug instead
of using pip and a requirements file. Since the repo is no longer supported
I/we would be happy to at least have the latest version in PyPI.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADXNPFG6ACEJPYZQWY3BMW3TCX5FLANCNFSM4F6PRFLA>
.
|
@Batev @veeralakrishna Please check the new release. |
Thank you for the New Release @SauceCat |
On line 251 in
pdp_plot_utils.py
, one of the parameters for_pdp_contour_plot()
iscontour_label_fontsize
and this causes the following error:According to the documentation for
matplotlib.pyplot.clabel()
, the parameter should be calledfontsize
.Source:
clabel()
documentationThe text was updated successfully, but these errors were encountered: