Skip to content

Configurable CPU rendering on per engine basis#9

Merged
dkyurtov merged 2 commits intorelease/3.29.0from
feature/configurable-cpu-rendering
Apr 29, 2025
Merged

Configurable CPU rendering on per engine basis#9
dkyurtov merged 2 commits intorelease/3.29.0from
feature/configurable-cpu-rendering

Conversation

@dkyurtov
Copy link

@dkyurtov dkyurtov commented Apr 9, 2025

No description provided.

	- Added the ability to enable cpu rendering for each
	  FlutterDartProject rather than globally.
	- Created ci.yml and release.yml
@dkyurtov dkyurtov force-pushed the feature/configurable-cpu-rendering branch 10 times, most recently from 3b72127 to b99ec6d Compare April 16, 2025 08:10
@dkyurtov dkyurtov force-pushed the feature/configurable-cpu-rendering branch from b99ec6d to 2139e86 Compare April 16, 2025 08:28
// Makes sure we have an existing pointer
auto iter = states_.find(pointer_data.device);
FML_DCHECK(iter != states_.end());
// FML_DCHECK(iter != states_.end());
Copy link
Member

Choose a reason for hiding this comment

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

Is this still needed?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, this assert crashing when you move the application to background.

// nullptr // release callback
// ));

// copy data, TODO: optimize to avoid repeated alloc/dealloc
Copy link
Member

Choose a reason for hiding this comment

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

I will suggest to create a followup task for the TODO.

Copy link
Author

Choose a reason for hiding this comment

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

I ll do that.

@dkyurtov dkyurtov merged commit 487053a into release/3.29.0 Apr 29, 2025
3 checks passed
@dkyurtov dkyurtov deleted the feature/configurable-cpu-rendering branch April 29, 2025 13:30
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.

3 participants