Skip to content

Long event names don't wrap in Tensorboard 61 #421

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
Closed
langmore opened this issue Aug 25, 2017 · 2 comments · Fixed by #479
Closed

Long event names don't wrap in Tensorboard 61 #421

langmore opened this issue Aug 25, 2017 · 2 comments · Fixed by #479

Comments

@langmore
Copy link

I have many long event names (they are node names), and Tensorboard 61 doesn't wrap these in the display. As a result, the names write over each other, and are unreadable.
@chihuahua
Copy link
Member

Thank you for reporting. Could you please include a screenshot to clarify? Do you mean tag names like these?

3gcm0ywunpc

@ro3c
Copy link

ro3c commented Sep 3, 2017 •
edited
Loading

Here is a sample of summary titles overlapping

image

Is there a work around for this other than much shorter names?

@chihuahua chihuahua mentioned this issue Sep 6, 2017
Merged
chihuahua added a commit that referenced this issue Sep 6, 2017
@chihuahua
Use flex positioning for card headings (#479)
c5a16f5 
In comments within #430, @wchargin proposed that we use flex positioning
to position elements within the header. That seems like a robust idea.

This PR thus uses @wchargin's proposed method to implement @jart's
original, space-efficient design of that makes use of both the left and
right sides of headings (if space is available).

This PR fixes the layout issues brought up in #430. This change also fixes
#421.
jart pushed a commit to jart/tensorboard that referenced this issue Sep 6, 2017
@chihuahua @jart
Use flex positioning for card headings (tensorflow#479)
f7102df 
In comments within tensorflow#430, @wchargin proposed that we use flex positioning
to position elements within the header. That seems like a robust idea.

This PR thus uses @wchargin's proposed method to implement @jart's
original, space-efficient design of that makes use of both the left and
right sides of headings (if space is available).

This PR fixes the layout issues brought up in tensorflow#430. This change also fixes
tensorflow#421.
jart pushed a commit that referenced this issue Sep 6, 2017
@chihuahua @jart
Use flex positioning for card headings (#479)
6dfa974 
In comments within #430, @wchargin proposed that we use flex positioning
to position elements within the header. That seems like a robust idea.

This PR thus uses @wchargin's proposed method to implement @jart's
original, space-efficient design of that makes use of both the left and
right sides of headings (if space is available).

This PR fixes the layout issues brought up in #430. This change also fixes
#421.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use flex positioning for card headings tensorflow/tensorboard
3 participants
@langmore @chihuahua @ro3c