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

projector: fix label/color legend #5974

Merged
merged 1 commit into from
Oct 17, 2022

Conversation

mromanelli9
Copy link
Contributor

Motivation for features / changes

Inside the projector plugin, a legend for labels/colors should appear when selecting a field from the "Color by" drop-down menu yet is not displayed.

Technical description of changes

The dom-if property is misconfigured for the vz-projector-legend template.

Screenshots of UI changes

Before:
before

After:
after

Detailed steps to verify changes work correctly (as executed by you)

  • Run the vz-projector in standalone mode
  • Use the default standalone_projector_config.json as config file (if run locally you need a simple webserver to serve the file via http)
  • Select the Iris tensor, then select "Class" under the "Color by" drop-down menu.
  • See the label/color legend appear just below

@qihach64 qihach64 requested a review from bmd3k October 13, 2022 20:09
Copy link
Contributor

@bmd3k bmd3k left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@bmd3k bmd3k merged commit 8dd5a3b into tensorflow:master Oct 17, 2022
@mromanelli9 mromanelli9 deleted the vz-projector-legend-fix branch October 18, 2022 07:17
qihach64 pushed a commit to qihach64/tensorboard that referenced this pull request Dec 19, 2022
## Motivation for features / changes

Inside the projector plugin, a legend for labels/colors should appear
when selecting a field from the "Color by" drop-down menu yet is not
displayed.

## Technical description of changes

The `dom-if` property is misconfigured for the `vz-projector-legend`
template.

Co-authored-by: Marco Romanelli <[email protected]>
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
## Motivation for features / changes

Inside the projector plugin, a legend for labels/colors should appear
when selecting a field from the "Color by" drop-down menu yet is not
displayed.

## Technical description of changes

The `dom-if` property is misconfigured for the `vz-projector-legend`
template.

Co-authored-by: Marco Romanelli <[email protected]>
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
## Motivation for features / changes

Inside the projector plugin, a legend for labels/colors should appear
when selecting a field from the "Color by" drop-down menu yet is not
displayed.

## Technical description of changes

The `dom-if` property is misconfigured for the `vz-projector-legend`
template.

Co-authored-by: Marco Romanelli <[email protected]>
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