Histogram with histtype='step' dows not respect lines.linewidth  #4065

Activity

1. tacaswell commented on Feb 6, 2015

   

   tacaswell

   on Feb 6, 2015

   Member

   More actions

   This is because hist use a Patch object to draw the stepped histogram, not a Line2D object.

   I would suggest not using hist and instead use np.histogram + ax.step

   vals, bins = np.histogram(..)
   ax.step(bins[:-1], vals, where='post')

   which should be aware of that rcparam.
2. tacaswell

   closed this as completedon Feb 6, 2015
3. maxnoe commented on Feb 6, 2015

   

   maxnoe

   on Feb 6, 2015

   ContributorAuthor

   More actions

   Ah ok. So it does respect patch.linewidth.
4. anntzer commented on Sep 17, 2015

   

   anntzer

   on Sep 17, 2015

   Contributor

   More actions

   I would like to suggest switching to using ax.step internally to plot such histograms, so that lines.linewidth gets respected (I think it makes sense e.g. for seaborn to set a relatively small patch.linewidth but a large lines.linewidth).
5. tacaswell commented on Sep 17, 2015

   

   tacaswell

   on Sep 17, 2015

   Member

   More actions

   @anntzer No, because then the return typo of hist would depend on a kwarg which is really bad form.
6. anntzer commented on Sep 17, 2015

   

   anntzer

   on Sep 17, 2015

   Contributor

   More actions

   Fair enough, reopened on seaborn's side as mwaskom/seaborn#704.
7. anntzer commented on Oct 8, 2015

   

   anntzer

   on Oct 8, 2015

   Contributor

   More actions

   Reporting an idea that had been shot down on seaborn's side:
   What about having hist locally patch rcParams to set patch.linewidth to lines.linewidth when making the plot? I still think the current behavior is unintuitive.
8. tacaswell commented on Oct 8, 2015

   

   tacaswell

   on Oct 8, 2015

   Member

   More actions

   👎 on that from me as well.

   If something like that were to happen, the right place would be for seaborn to wrap hist and then use the style context manager to locally over-ride the rcparams.