ORNL-AMO / AMO-Tools-Desktop

AMO-Tools-Desktop is an energy efficiency calculation application for use with industrial equipment such as pumps, furnaces, fans, and motors, as well as for industrial systems such as steam. Currently in beta.
Other
37 stars 21 forks source link

Sankey Print #5339

Closed koay9f closed 2 years ago

koay9f commented 2 years ago

has weird lines image.png

image.png

nbintertech commented 2 years ago

Dropping this for the moment to work on other priority.

Only hunch is that there are some Plotly initialization animations which are smearing. Possibly a 'smearing' from the text style or shadow? Looks like sankey prints clean in both VERIFI and MEASUR web app but not desktop app wrapped w/ electron. Look into changing electron print for sankey or general

Tried: x - chart seems to be fully rendered and document ready status complete before printing is done x - turn off animations at CSS level. Works for rendered plot but not printed sankey x - try delaying print timer x - force a window focus on page x - try checking document on ready from window service x- hide labels if print view and put the labels into a table

Look at...

https://gist.github.com/fcingolani/3300351 https://stackoverflow.com/questions/49650784/printing-a-pdf-file-with-electron-js https://www.electronjs.org/docs/latest/api/web-contents

nbintertech commented 2 years ago

branch 'upgrade-plotly' can be used for debugging this