-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
404 on some images and fonts behind proxy #1885
Comments
I found the webfiles.zip in the python package installation and copied over font-roboto and tf-interactive-inference-dashboard to the root website so that nginx can serve these static files. But there is also a trace_viewer_index.html and I'm not sure if it is ok for nginx to serve that too. |
Thanks for the report. The You can try this out by uninstalling tensorboard and installing our nightly build If for some reason that doesn't fix the issue, comment here and we can reopen and investigate. |
Can repro on 1.13.0: tensorboard/third_party/fonts.bzl Line 425 in 5c384ac
|
Hmm, thanks for catching this. Maybe we can just omit the leading / for those urls? |
Any progress on this issue? Seem still not resolved in 2.0.1 |
The issue with font remains in 2.2.0. Also the profile plugin doesn't respect |
This impacts jupyter_tensorboard integration as well. I worked around it for jupyter_tensorboard in cliffwoolley/jupyter_tensorboard@b351bb4 by making a static route for the lspvic/jupyter_tensorboard#63 includes my workaround in the meanwhile. |
Hello, we also have the issue in our tensorboard integration at Huggingface. |
+1 |
Do you have any updates on this issue? |
This continues to be an issue. For static assets it appears to be a relatively simple fix to The fix would likely be much easier for a Googler - I think it would take some time for an external dev to figure out and setup all the Bazel stuff. |
I am using tensorboard behind a proxy at www.domain.com/tensorboard and it is making requests to paths such as www.domain.com/font-roboto/... and www.domain.com/tf-interactive-inference-dashboard/...
Can these be changed to have the /tensorboard prefix? FWIW, the --path_prefix option does not add the prefix to these urls.
For bug reports, please include the following:
tensorboard
): 1.12.2Please describe the bug as clearly as possible, and if possible provide a minimal example (code, data, and/or command line) to reproduce the issue. Thanks!
The text was updated successfully, but these errors were encountered: