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
[python] added plot_split_value_histogram function #2043
Conversation
python-package/lightgbm/plotting.py
Outdated
xlim = (bins[0] - range_result * 0.2, bins[-1] + range_result * 0.2) | ||
ax.set_xlim(xlim) | ||
|
||
ax.set_yticks([int(tick) for tick in ax.get_yticks()]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any matplotlib expert here? 😃
I'm not sure how to do this properly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping @henry0312 @chivee
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems that ax.yaxis.set_major_locator(MaxNLocator(integer=True))
is right way to do it.
https://stackoverflow.com/a/34880501
https://stackoverflow.com/a/11417609
https://stackoverflow.com/a/38096332
https://www.scivision.dev/matplotlib-force-integer-labeling-of-axis/
@henry0312 can you review this PR? thanks! |
Sure, but I'm on a trip now, so I'll review in two days 🙋♂️
2019年4月30日(火) 23:19 Guolin Ke <notifications@github.com>:
@henry0312 <https://github.com/henry0312> can you review this PR? thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2043 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADJ2GQJEPUO6J7KBOX7F3LPTBINJANCNFSM4G43U53Q>
.
--
Tsukasa OMOTO
tomoto@if-el.se
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The function looks helpful ;)
No description provided.