Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Open
Description
Whilempl_to_plotly is little known and receives little love, this bug is pretty easy to fix. Would you be open to a PR?
Details
importmatplotlib.pyplotaspltfromplotly.toolsimportmpl_to_plotlyfromplotly.offlineimportplot
In matplotlib this produces a barplot with labels:
labels= ['a','b','c']values= [1,2,3]f=plt.figure(figsize=(6,4))plt.bar(labels,values)plt.tight_layout()
But conversion to plotly looses the labels:
plotly_fig = mpl_to_plotly(f)plot(plotly_fig)
A minimal fix would be to modifyprep_ticks by appending:
ifaxis_dict.get("type")=="date":returnaxis_dictvals= []texts= []fortickinaxis.majorTicks:vals.append(tick.get_loc())texts.append(tick.label1.get_text())iftexts:axis_dict= {}axis_dict['tickmode']='array'axis_dict['tickvals']=valsaxis_dict['ticktext']=textsreturnaxis_dict
which produces:
prep_ticks is defined in:
plotly.py/plotly/matplotlylib/mpltools.py
Lines 428 to 514 inc54a2bd
| defprep_ticks(ax,index,ax_type,props): | |
| """Prepare axis obj belonging to axes obj. | |
| positional arguments: | |
| ax - the mpl axes instance | |
| index - the index of the axis in `props` | |
| ax_type - 'x' or 'y' (for now) | |
| props - an mplexporter poperties dictionary | |
| """ | |
| axis_dict=dict() | |
| ifax_type=="x": | |
| axis=ax.get_xaxis() | |
| elifax_type=="y": | |
| axis=ax.get_yaxis() | |
| else: | |
| returndict()# whoops! | |
| scale=props["axes"][index]["scale"] | |
| ifscale=="linear": | |
| # get tick location information | |
| try: | |
| tickvalues=props["axes"][index]["tickvalues"] | |
| tick0=tickvalues[0] | |
| dticks= [ | |
| round(tickvalues[i]-tickvalues[i-1],12) | |
| foriinrange(1,len(tickvalues)-1) | |
| ] | |
| ifall([dticks[i]==dticks[i-1]foriinrange(1,len(dticks)-1)]): | |
| dtick=tickvalues[1]-tickvalues[0] | |
| else: | |
| warnings.warn( | |
| "'linear' {0}-axis tick spacing not even, " | |
| "ignoring mpl tick formatting.".format(ax_type) | |
| ) | |
| raiseTypeError | |
| except (IndexError,TypeError): | |
| axis_dict["nticks"]=props["axes"][index]["nticks"] | |
| else: | |
| axis_dict["tick0"]=tick0 | |
| axis_dict["dtick"]=dtick | |
| axis_dict["tickmode"]=None | |
| elifscale=="log": | |
| try: | |
| axis_dict["tick0"]=props["axes"][index]["tickvalues"][0] | |
| axis_dict["dtick"]= ( | |
| props["axes"][index]["tickvalues"][1] | |
| -props["axes"][index]["tickvalues"][0] | |
| ) | |
| axis_dict["tickmode"]=None | |
| except (IndexError,TypeError): | |
| axis_dict=dict(nticks=props["axes"][index]["nticks"]) | |
| base=axis.get_transform().base | |
| ifbase==10: | |
| ifax_type=="x": | |
| axis_dict["range"]= [ | |
| math.log10(props["xlim"][0]), | |
| math.log10(props["xlim"][1]), | |
| ] | |
| elifax_type=="y": | |
| axis_dict["range"]= [ | |
| math.log10(props["ylim"][0]), | |
| math.log10(props["ylim"][1]), | |
| ] | |
| else: | |
| axis_dict=dict(range=None,type="linear") | |
| warnings.warn( | |
| "Converted non-base10 {0}-axis log scale to 'linear'""".format(ax_type) | |
| ) | |
| else: | |
| returndict() | |
| # get tick label formatting information | |
| formatter=axis.get_major_formatter().__class__.__name__ | |
| ifax_type=="x"and"DateFormatter"informatter: | |
| axis_dict["type"]="date" | |
| try: | |
| axis_dict["tick0"]=mpl_dates_to_datestrings(axis_dict["tick0"],formatter) | |
| exceptKeyError: | |
| pass | |
| finally: | |
| axis_dict.pop("dtick",None) | |
| axis_dict.pop("tickmode",None) | |
| axis_dict["range"]=mpl_dates_to_datestrings(props["xlim"],formatter) | |
| ifformatter=="LogFormatterMathtext": | |
| axis_dict["exponentformat"]="e" | |
| returnaxis_dict |