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

tf.summary.text option to control whether text is treated as markdown #3590

Open
drdozer opened this issue May 5, 2020 · 3 comments
Open

Comments

@drdozer
Copy link

drdozer commented May 5, 2020

I'm exporting text summaries from keras into tensorboard. The text contains various markdown sequences, but I don't intend this to be interpreted as markdown. However, tensorboard insists on trying to treat it as markdown. In the best case this gives entertainingly broken rendering in the text viewer, and worst case it causes the server to bomb in the text plugin. The text I'm exporting should never be treated as markdown. It isn't markdown.

Can we please have some way to tag a text summary as being not markdown? Say::

tf.summary.text('sometext', text_tensor, step, is_raw_text=True)

And then have this communicated through to tensorboard, and have tensorboard not attempt to treat it as markdown?

@nfelt
Copy link
Contributor

nfelt commented May 5, 2020

Duplicate of #830

@nfelt nfelt marked this as a duplicate of #830 May 5, 2020
@nfelt
Copy link
Contributor

nfelt commented May 5, 2020

Thanks for the request; there is an existing very similar feature request at #830, except that it talks about making this an option in the UI rather than an option at the summary write time.

Would a UI option work in your case? It's a bit more flexible and more easily compatible with cases when the tf.summary call is inside a library or some other place that isn't directly in control of the user.

@nfelt
Copy link
Contributor

nfelt commented Oct 27, 2021

FWIW, #5378 implemented a UI option to toggle markdown rendering off (sticky for a given hostname/port). I'll rescope this FR to be just about having a summary-level control, which could still be helpful if users want either a mix of markdown and not-markdown, or want to ensure the data is always treated a certain way regardless of the setting's default value.

@nfelt nfelt changed the title Dissable markdown processing of text tf.summary.text option to indicate whether text should be treated as markdown Oct 27, 2021
@nfelt nfelt changed the title tf.summary.text option to indicate whether text should be treated as markdown tf.summary.text option to control whether text is treated as markdown Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants