Skip to content
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

Propagate Plotly.js console errors to Python #4903

Open
ndrezn opened this issue Nov 26, 2024 · 1 comment
Open

Propagate Plotly.js console errors to Python #4903

ndrezn opened this issue Nov 26, 2024 · 1 comment
Labels
feature something new P3 backlog

Comments

@ndrezn
Copy link
Member

ndrezn commented Nov 26, 2024

Occasionally Plotly.js console errors are relevant to Python developers, for example in: plotly/plotly.js#7289 where we show an error about a missing style.

For Python developers (especially in Jupyter) this is really gnarly to debug because they're never shown the issue unless they can figure out to check the console somehow.

I would wager there are a class of JS errors that we might not catch like this but I might be wrong. @alexcjohnson do you know if we have any handling in place for this kind of thing already?

@gvwilson gvwilson added feature something new P3 backlog labels Nov 26, 2024
@alexcjohnson
Copy link
Collaborator

I’m not sure we want to go down this road. Most JS errors are simply bugs, those few where we’re trying to give the user some info shouldn’t we be able to handle before it leaves Python?

The other one is to expose Plotly.validate to Python users, that’s able to catch more than Python-side validation because it happens after supplyDefaults so it can tell you about interactions between different attributes (only reference axes that exist, only set marker attributes if you’re showing markers, that sort of thing). Do we already have an issue for that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new P3 backlog
Projects
None yet
Development

No branches or pull requests

3 participants