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

fix(am-dbg): dynamic rendering for log view #98

Open
pancsta opened this issue Aug 1, 2024 · 3 comments
Open

fix(am-dbg): dynamic rendering for log view #98

pancsta opened this issue Aug 1, 2024 · 3 comments
Labels
am-dbg tools/am-dbg bug Something isn't working p:3 Low priority

Comments

@pancsta
Copy link
Owner

pancsta commented Aug 1, 2024

am-dbg becomes less responsive after a longer heavy streaming. It could be either UI or data parsing leak.

Steps:

  • start the pubsub simulator for 10min with 20 peers and 30 topics (defaults)
  • keep it in the tail mode until it finishes
  • go back / forth

Result:

  • noticeable delay

Workarounds:

  • export & --import-data
  • switch to another machine and back
@pancsta pancsta added bug Something isn't working am-dbg tools/am-dbg labels Aug 1, 2024
@pancsta pancsta changed the title am-dbg becomes less responsive after a longer heavy streaming am-dbg: UI becomes less responsive after a longer heavy streaming Aug 1, 2024
@pancsta pancsta added the p:1 High priority label Aug 1, 2024
@pancsta
Copy link
Owner Author

pancsta commented Aug 8, 2024

LogView seems to be the issue, could use caching or simply a periodic restart (eg every 1000 cursor moves).
cpu dark

@pancsta
Copy link
Owner Author

pancsta commented Aug 8, 2024

Additionally there seem to be a leak in tcell (*CellBuffer) Resize, as visible on this dump after 10min of streaming.
mem hot

Repro: focus switching is very slow

@pancsta
Copy link
Owner Author

pancsta commented Aug 9, 2024

Disabling word wrap fixes most of the issues.

TODO:

Steps:

  • load the sim dump
  • hold Tab to rotate focus

Result: focus change is slow for sim, but fast for eg sim-p0

@pancsta pancsta changed the title am-dbg: UI becomes less responsive after a longer heavy streaming fix(am-dbg): dynamic rendering for log view Nov 23, 2024
@pancsta pancsta added p:3 Low priority and removed p:1 High priority labels Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
am-dbg tools/am-dbg bug Something isn't working p:3 Low priority
Projects
None yet
Development

No branches or pull requests

1 participant