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

Show FPS directly on screen #274

Open
Cryolitia opened this issue Sep 24, 2018 · 3 comments
Open

Show FPS directly on screen #274

Cryolitia opened this issue Sep 24, 2018 · 3 comments

Comments

@Cryolitia
Copy link

Thanks for you work, but could you please show the FPS on the screen instead of stdout.
My English is poor, please don't mind.

@rom1v rom1v changed the title It's not a bug,but i want it to work better Show FPS directly on screen Sep 24, 2018
@rom1v
Copy link
Collaborator

rom1v commented Sep 24, 2018

Yes, this would be better, but requires some work.

Firstly, the dependency SDL_ttf must be added to be able to draw text. This would allow to write an OSD component, which could be used to print various messages.

Then, we need to refresh (compute+render) the FPS between frames. Frames are sent only when the device surface content changes, so there may be seconds or minutes without any screen refresh. The FPS should be updated in that case.

By the way (just a side note), the FPS is just a hint to get an idea of the performance, but it may be misleading: if you watch a 25 FPS fullscreen video, then FPS will be 25, no more, which is perfect. On the other hand, if the encoding/decoding/rendering is not fast enough, the FPS can be 55 while the screen renders at 60 fps, in that case the framerate is too low. Just to say that the raw FPS is not meaningful in itself.

@Cryolitia
Copy link
Author

Could it be easier by showing it on window's title bar, it seems to be like a good idea.

@AndroidDeveloperLB
Copy link

Can you please allow to show it in either the title bar or the screen?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants