KEMBAR78
scalar: manually set background to transparent by japie1235813 · Pull Request #5567 · tensorflow/tensorboard · GitHub
Skip to content

Conversation

@japie1235813
Copy link
Contributor

The background of charts in scalar card have black backgrounds. It is because of the threejs upgrades 420916a. Before we find a long-term solution, we can set the background opacity to zero as the temporary fix

before
Screen Shot 2022-02-14 at 4 21 45 PM

after
Screen Shot 2022-02-14 at 4 22 58 PM

@japie1235813 japie1235813 requested a review from bmd3k February 15, 2022 00:25
alpha: true,
} as WebGLContextAttributes) as WebGLRenderingContext,
});
// Workaround to fix background transparency is not set propoerly in threejs WebGLRenderer.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which property was supposed to make the background transparent? Is it the "alpha: true"? Can you mention that?

alpha: true,
} as WebGLContextAttributes) as WebGLRenderingContext,
});
// Workaround to fix background transparency is not set propoerly in threejs WebGLRenderer.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nite: propoerly => properly.

@japie1235813
Copy link
Contributor Author

From the reply of three.js developers. mrdoob/three.js#23495
The alpha and antialias params should be moved out of context.
https://github.com/mrdoob/three.js/blob/766b95bad6ae2c98c5bf1c4c775c7c9e8f72f2d2/src/renderers/WebGLRenderer.js#L62

Also remove precision cause it is not WebGLContextAttributes
https://developer.mozilla.org/en-US/docs/Web/API/WebGLRenderingContext/getContextAttributes

@japie1235813 japie1235813 merged commit 806dd3e into tensorflow:master Feb 15, 2022
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
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.

2 participants