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

Logging level too low when running rerun-cli --serve-web #9024

Open
jleibs opened this issue Feb 12, 2025 · 2 comments
Open

Logging level too low when running rerun-cli --serve-web #9024

jleibs opened this issue Feb 12, 2025 · 2 comments
Labels
🦟 regression A thing that used to work in an earlier release

Comments

@jleibs
Copy link
Member

jleibs commented Feb 12, 2025

0.21:

➜  rerun --serve-web
[2025-02-12T18:33:49Z INFO  re_sdk_comms::server] Hosting a SDK server over TCP at 0.0.0.0:9876. Connect with the Rerun logging SDK.
[2025-02-12T18:33:49Z INFO  re_ws_comms::server] Hosting a WebSocket server on ws://localhost:9877. You can connect to this with a native viewer (`rerun ws://localhost:9877`) or the web viewer (with `?url=ws://localhost:9877`).
[2025-02-12T18:33:49Z INFO  re_sdk::web_viewer] Hosting a web-viewer at http://localhost:9090?url=ws://localhost:9877

0.22:

➜  rerun --serve-web

Having the INFO log statements with port number, etc are extremely important in this case.

@jleibs jleibs added the 🦟 regression A thing that used to work in an earlier release label Feb 12, 2025
@Wumpf
Copy link
Member

Wumpf commented Feb 12, 2025

this was a deliberate change we made to reduce logspam in the context of the viewer. We have to find a compromise of when to show how much log

@jleibs
Copy link
Member Author

jleibs commented Feb 12, 2025

Yeah, the trade-off is hard.

That said, the real motivation for the change (#7815) wasn't to reduce logspam in the viewer, but logspam via the SDK when using in a notebook context.

I don't actually see an issue with the viewer sticking with INFO by default. It's the SDK info messages that are kind of spammy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦟 regression A thing that used to work in an earlier release
Projects
None yet
Development

No branches or pull requests

2 participants