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

perf: Double rendering #10980

Open
mmeeks opened this issue Jan 21, 2025 · 1 comment
Open

perf: Double rendering #10980

mmeeks opened this issue Jan 21, 2025 · 1 comment
Assignees
Labels
24.04 enhancement New feature or request performance Improving COOL performance unconfirmed

Comments

@mmeeks
Copy link
Contributor

mmeeks commented Jan 21, 2025

Open a spreadsheet, hit page-down a few times - notice some tiles have been rendered twice.

We should save that double rendering. I believe this is because we send a tilecombine that is interpreted as being a request to force rendering of a tile - and the JS client shoudl not have that 'power' - it should be sent a cached version.

Meanwhile DocumentBroker needs a way to force a new key-frame, and I tihnk the JS does this by accident, and sometimes these race unhelpfully.

So the 'oldwid=' thing needs adapting / filtering from the JS side to avoid this in the Kit/ code =)

Thanks !

@mmeeks
Copy link
Contributor Author

mmeeks commented Jan 21, 2025

You can see the count of tile rendering in the grid overlay over the tiles with details there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
24.04 enhancement New feature or request performance Improving COOL performance unconfirmed
Projects
Status: No status
Development

No branches or pull requests

2 participants