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

Trace uid not generated #765

Closed
dmossie opened this issue Oct 10, 2018 · 5 comments
Closed

Trace uid not generated #765

dmossie opened this issue Oct 10, 2018 · 5 comments

Comments

@dmossie
Copy link

dmossie commented Oct 10, 2018

Hey there,

When I create a new trace using plotly.js@1.41.3, it is not generated with a uid.
I downgraded to plotly.js@1.35.2 and it worked fine.

I am not sure if this is a compatibility issue or a plotly.js issue.

@dmt0
Copy link
Contributor

dmt0 commented Oct 10, 2018

This change was introduced in 1.39 of plotly.js. You can see the motivation for it in this PR

@dmossie
Copy link
Author

dmossie commented Nov 22, 2018

Hey @dmt0 - thanks for your response. Do you know if it is possible to get them to show up again? I want to upgrade to the newest version of all the plotly packages, but I rely on the trace UIDs for some business specific customization's I have made.

@dmt0
Copy link
Contributor

dmt0 commented Nov 22, 2018

You can't really get them in data, but they are still in fullData.

@dmossie
Copy link
Author

dmossie commented Dec 4, 2018

Thanks @dmt0
Do you know how to go about accessing fullData?
Also - is it possible to access outside the editor - such as using the onUpdate prop?

@dmt0
Copy link
Contributor

dmt0 commented Dec 4, 2018

graphDiv._fullData
Where graphDiv is the reference to the

where plotly.js lives.
Basically it's a structure that's in plotly.js library, not react-chart-editor, so yes, you can access it anywhere.

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

No branches or pull requests

3 participants