Skip to content

Fix the chrome install issue#423

Merged
wdecoster merged 3 commits intowdecoster:masterfrom
microbemarsh:kaleido_update
Aug 19, 2025
Merged

Fix the chrome install issue#423
wdecoster merged 3 commits intowdecoster:masterfrom
microbemarsh:kaleido_update

Conversation

@microbemarsh
Copy link
Contributor

My machines never ran into the issues here #422 but it does make sense that others might have issue with that and I felt partially responsible to help. In plot.py I swapped out kaleido.get_chrome_sync() for pio.get_chrome() which I believe will install chrome in the user conda env (added some error warnings just in case). Minimal test performed and all the output files expected were seen.

@lsterck I hope this is able to fix your issue as I wasn't able to replicate it fully :/

@microbemarsh microbemarsh marked this pull request as ready for review August 19, 2025 02:51
@lsterck
Copy link

lsterck commented Aug 19, 2025

Hi @microbemarsh

thanks for picking up on this!
I personally also don't have any issues installing it on local servers (well, I was able to add a fix for the issue on our local servers ;-) ) but the issue is mainly when deploying via conda in a galaxy context, where the conda-env is more restrictive I understood and I, for obvious reasons, can't do what I did with my local fix.

@wdecoster
Copy link
Owner

wdecoster commented Aug 19, 2025

Thanks @microbemarsh! Hope this fixes the issue that @lsterck encountered, do let us know if problems persist!
(now available as v1.46.1)

@wdecoster wdecoster merged commit d6adee0 into wdecoster:master Aug 19, 2025
1 check passed
@lsterck
Copy link

lsterck commented Aug 20, 2025

Great , thanks both !

my local test using the conda install does seem to pass now indeed (while failing before on the reported issue)
I do get this in the stderr though:

"WARNING\troot:plot.py:()- Plotly could not fetch or find Chrome automatically. Static exports may fail unless BROWSER_PATH is set. Details: [Errno 13] Permission denied: '/usr/local/lib/python3.13/site-packages/choreographer/cli/browser_exe'

Is that something to worry about? or that I can/should amend?
Or can I safely ignore if no static images are requested?

@wdecoster
Copy link
Owner

Well... let us know :-D

@lsterck
Copy link

lsterck commented Aug 21, 2025

seems to be fine.
thanks!

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.

3 participants