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

session_log does not contain Statistics information #2468

Open
chutoutian opened this issue Oct 21, 2024 · 5 comments
Open

session_log does not contain Statistics information #2468

chutoutian opened this issue Oct 21, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@chutoutian
Copy link

chutoutian commented Oct 21, 2024

Hi I am using ALVR20.11.1 and I want to use the history statistics like latency and bitrate but the logs say:

11:41:31.230 INFO Client 8428.client: [Warning] Statistics summary not ready!

And the [STATS] and [GRAPH] are empty:

11:41:37.946 [STATS]
11:41:37.946 [TRACKING] {"device_motions":[["/user/hand/left",{"pose":{"orientation":[-0.05758141,0.21218827,0.33082303,-0.91772366],"position":[-0.30668914,0.6977202,-0.86611617]},"linear_velocity":[0.0,0.0,0.0],"angular_velocity":[0.0,0.0,0.0]}],["/user/hand/right",{"pose":{"orientation":[-0.16810232,-0.2498652,-0.524207,-0.7965652],"position":[0.017634995,0.57615304,-0.29410082]},"linear_velocity":[0.0,0.0,0.0],"angular_velocity":[0.0,0.0,0.0]}],["/user/head",{"pose":{"orientation":[0.03658971,0.18495813,0.03534971,0.9814287],"position":[0.024379883,1.0507616,-0.18076341]},"linear_velocity":[0.0,0.0,0.0],"angular_velocity":[0.0,0.0,0.0]}]],"hand_skeletons":[null,null],"eye_gazes":[null,null],"fb_face_expression":null,"htc_eye_expression":null,"htc_lip_expression":null}
11:41:37.946 [GRAPH]

Are there some configurations I missed?

Thanks

@zmerp
Copy link
Member

zmerp commented Oct 21, 2024

Oh I did not fully realized i did that. But at the same time, I this that's fine. statistics are quite spammy and it would be better to leave it out of logging. logs are meant to be human readable, and a bunch of floating point values are not easy to understand. if you intend to use the statistics data you should instead subscribe to the events websocket at ws://localhost:8082/api/events

@chutoutian
Copy link
Author

Hi, thanks. I got the wanted statistics. BTW, current logs still contain the user's pose data which has a similar format to the statistics. But anyway, thanks~~

@zmerp
Copy link
Member

zmerp commented Oct 22, 2024

You're right. That should be more consistent

@The-personified-devil
Copy link
Collaborator

You're right. That should be more consistent

Tbh I don't think so. Logging is a huge mess and intentionally introducing spam is seems like a bad idea even if we have the log enabling disabling choices. It seems good as is as the statistics information are already accessible for debugging.
For the future, if anything, I'd say to also externalize pose information if that can be done in an easily accessible way (maybe wmr already)

@The-personified-devil
Copy link
Collaborator

Especially because actually structured information is way more useful than trying to parse info from text logs

@zmerp zmerp added the enhancement New feature or request label Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants