-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
statsviz.TimeSeries.GetValue is called multiple times per second #119
Comments
Currently statsviz creates a new connection for each client/browser that connects to it. The orignal use case wasn't to have more than one client. |
I was sure it was a feature and built our own locking around it 😆 |
Bugs, features... That's just different point of views 😅😅 But yes I think that statsviz should be restructured so that there's a single timer for N client connections. Shouldn't be too hard |
Open 2 or more pages and you will see the phenomenon
code like this:
Here's the log with 3 pages opened
![image](https://private-user-images.githubusercontent.com/43053461/292166877-5e5b6956-abef-4a9b-ba16-8d390ac3c694.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkyNDQ0MzAsIm5iZiI6MTczOTI0NDEzMCwicGF0aCI6Ii80MzA1MzQ2MS8yOTIxNjY4NzctNWU1YjY5NTYtYWJlZi00YTliLWJhMTYtOGQzOTBhYzNjNjk0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjExVDAzMjIxMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTZkNmU0MDgyOGRlNzcwNWVjNDFkYTBhNTQzZjA2ZDFlMzg0NDk5NTk5MWIyNjQ4MGQ5NWMyM2M2NTI1N2Y1MDUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.ufJ1vKPdc30E73kGDvLsjBKBQAOuSE0q42wklSiYjRg)
The text was updated successfully, but these errors were encountered: