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.

Sign up for GitHub

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

Jump to bottom

plot.show() does not work on Linux snap Firefox #170

Closed
MarkSwanson opened this issue Nov 5, 2023 · 3 comments · Fixed by #217
Closed

plot.show() does not work on Linux snap Firefox #170

MarkSwanson opened this issue Nov 5, 2023 · 3 comments · Fixed by #217
Assignees
@andrei-ng
Labels
good first issue Good for newcomers

Comments

@MarkSwanson
Copy link

snap Firefox has its own /tmp directory now. snap Firefox no longer sees the real /tmp directory - so it can't find the plotly generated HTML file.
Firefox 119.0
Ubuntu 22.04

Possible solution: plotly already provides plot.write_html(filename) - so just provide a plot.show(filename) ?

Thanks!
@andrei-ng
Copy link
Collaborator

@MarkSwanson, yes, you are correct. With snap, even most (all) of the examples won't work.

I will look into it.

@andrei-ng andrei-ng self-assigned this Jul 22, 2024
@andrei-ng andrei-ng added the good first issue Good for newcomers label Jul 22, 2024
andrei-ng added a commit that referenced this issue Jul 30, 2024
@andrei-ng
provide alternative show_html method
e3e615a 
 - the show() method by default creates a random named file in `/tmp`
   dir, however in some cases `/tmp` is not available, e.g., snap's Firefox
   runs in an isolated enviroment and cannot access the system's `/tmp`
 - provide a method that calls write_html followed by show for easier
   access

Closes #170

Signed-off-by: Andrei Gherghescu <[email protected]>
@andrei-ng andrei-ng mentioned this issue Jul 30, 2024
Merged
@andrei-ng
Copy link
Collaborator

andrei-ng commented Jul 30, 2024 •
edited
Loading

@MarkSwanson , I added a straightforwad implementation in #217. Is this what you had in mind as well?
I actually used it as I got bit by snap as well :)

@andrei-ng
Copy link
Collaborator

andrei-ng commented Jul 31, 2024 •
edited
Loading

This implementation will be available after the PR is merged.

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

@andrei-ng andrei-ng

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add show_html(filename) method plotly/plotly.rs
2 participants
@MarkSwanson @andrei-ng