Skip to content

Desktop: Directly upload frame buffer #2930

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

Merged
merged 4 commits into from
Jul 25, 2025
Merged

Conversation

TrueDoctor
Copy link
Member

Bypass copying the frame buffer data to an intermediate vec for improved performance.
image

@timon-schelling
Copy link
Collaborator

after disabling gpu the correct way, I get perfect frames and chromium browser level performance and responsiveness.

window_size_sender: Sender<WindowSize>,
_viewport_frame_buffer: Option<FrameBuffer>,
_viewport_frame_buffer: Option<wgpu::Texture>,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will not be used In the future right?
I think you should remove both frame buffers

@adamgerhant adamgerhant changed the title Directly upload frame buffer Desktop: Directly upload frame buffer Jul 25, 2025
@TrueDoctor TrueDoctor force-pushed the directly_upload_frame_buffer branch from 5cce137 to 7b4c54d Compare July 25, 2025 09:48
@TrueDoctor TrueDoctor force-pushed the directly_upload_frame_buffer branch from 7b4c54d to c64f4c1 Compare July 25, 2025 09:50
@TrueDoctor TrueDoctor merged commit a52ee70 into master Jul 25, 2025
4 checks passed
@TrueDoctor TrueDoctor deleted the directly_upload_frame_buffer branch July 25, 2025 12:04
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