Skip to content

switch from graph_objects to px#20

Merged
tdhopper merged 2 commits intotdhopper:masterfrom
nicolaskruchten:px
Sep 17, 2020
Merged

switch from graph_objects to px#20
tdhopper merged 2 commits intotdhopper:masterfrom
nicolaskruchten:px

Conversation

@nicolaskruchten
Copy link
Copy Markdown
Contributor

Let's see if Kaleido works "out of the box" here with the rendering framework! in principle there's no need for the Image(fig.to_image(format="png", width=900, height=900)) stuff :)

@tdhopper
Copy link
Copy Markdown
Owner

@nicolaskruchten Awesome! Does this look right? 5f636f0dda78390aa9c727b6--pythonplot-com.netlify.app

@nicolaskruchten
Copy link
Copy Markdown
Contributor Author

Looks as intended!

@nicolaskruchten
Copy link
Copy Markdown
Contributor Author

I might suggest rendering smaller than 900x900 as the as the fonts look really small... I can tack on a couple of commits here if the overall approach is right :)

@tdhopper
Copy link
Copy Markdown
Owner

@nicolaskruchten sure

@tdhopper tdhopper merged commit f5fcc4b into tdhopper:master Sep 17, 2020
@nicolaskruchten
Copy link
Copy Markdown
Contributor Author

Awesome! Thanks, this lets PX shine at what it's good at ;) PX initially stood for Plotly Explore actually but we changed it to Express because it's not really just for EDA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants