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

Tensorboard fails to open if tensorflow prints warnings #605

Open
RalphJO opened this issue Oct 8, 2024 · 2 comments
Open

Tensorboard fails to open if tensorflow prints warnings #605

RalphJO opened this issue Oct 8, 2024 · 2 comments

Comments

@RalphJO
Copy link
Contributor

RalphJO commented Oct 8, 2024

Hello,

In tensorflow:::tensorboard_version(), if tensorflow gives any warnings the call will fail with Error: invalid version specification '<tensorflow warning text>'

This comes from this line of the code:

ver <- package_version(system("tensorboard --version_tb", intern = TRUE))

Please could you consider adding an ignore.stderr = TRUE to the system call so that warnings from tensorflow are not parsed, only the stdout from tensorboard?

ver <- package_version(system("tensorboard --version_tb", intern = TRUE, ignore.stderr = TRUE)) 

Thank you,

Ralph

@t-kalinowski
Copy link
Member

Thanks for the bug report! Would you like to submit a PR?

@RalphJO
Copy link
Contributor Author

RalphJO commented Oct 8, 2024

PR submitted, thanks.

#606

t-kalinowski added a commit that referenced this issue Oct 8, 2024
Issue #605 - ignore stderr when parsing tensorboard version
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

No branches or pull requests

2 participants