Add support for new plot types: polar, radar, 3D scatter, bar, and symbolic axes#30
Conversation
| axis.add_option("zmajorgrids", "false") | ||
|
|
||
| # Title | ||
| if hasattr(figure_layout.scene, "title") and getattr(figure_layout.scene.title, "text", None): |
There was a problem hiding this comment.
The attribute fig.layout.scene.title does not seem to exist. Do you have an example of code where this is used?
| if options is not None: | ||
| code += f"[{options}]" | ||
| else: | ||
| code += "[only marks]" |
There was a problem hiding this comment.
is there a particular reason for this option only marks? for a figure with mode=markers+lines, it adds this option while it shouldn't
| symbolic_theta = list(dict.fromkeys(theta)) | ||
|
|
||
| if angular_categoryorder is not None: | ||
| if angular_categoryorder == "category ascending": |
There was a problem hiding this comment.
@jsaussereau do you have example of plotly figure where this attribute is used ? I did not manage to find such one
|
@jsaussereau As you can see in #33 there were issues with complex texts in symbolic expression. My fix is not perfect, but it works so far. |
TODO before merging
@jsaussereau Thanks again for your contribution :D |
2 participants
Summary
This pull request introduces support for more plot types
Main changes
Testing
Test were added for each new plot type using examples from plotly documentation