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

Jringle/fps debug #63

Merged
merged 3 commits into from
Apr 30, 2024
Merged

Conversation

ringlej
Copy link
Contributor

@ringlej ringlej commented Mar 19, 2024

@axelson This will log every second the number of frames rendered.
Could you run this against the code you are having issues with?
Also with SCENIC_DRIVER_LOCAL=glfw for comparison against the non-cairo code on host,
and SCENIC_DRIVER_LOCAL=drm for comparison against the non-cairo code on rpi3

On the application config for the viewport set the debug_fps option to 1 or 2 to get debug fps output:

config :hello_scenic_full, :viewport,
  name: :main_viewport,
  size: {800, 600},
  theme: :dark,
  default_scene: HelloScenicFull.Scene.Components,
  drivers: [
    [
      module: Scenic.Driver.Local,
      name: :local,
      window: [resizeable: false, title: "hello_scenic_full"],
      on_close: :stop_system,
      debug_fps: 2,
      layer: 1,
      opacity: 180,
    ]
  ]

ringlej added 2 commits March 18, 2024 20:23
The STDIO_TIMEOUT returns millisecond units, so the STDIO_TIMEOUT
should also be in millisecond units
@ringlej
Copy link
Contributor Author

ringlej commented Mar 19, 2024

@crertel Please re-review. I added another commit

@ringlej ringlej requested a review from crertel March 19, 2024 02:38
@ringlej ringlej force-pushed the jringle/fps-debug branch from 2c6c51a to 82c4e64 Compare March 19, 2024 02:43
@ringlej ringlej force-pushed the jringle/fps-debug branch from 82c4e64 to 9ae099b Compare March 19, 2024 02:49
@ringlej ringlej mentioned this pull request Apr 12, 2024
@crertel crertel merged commit ca207a8 into ScenicFramework:main Apr 30, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants