Skip to content

Generalize the GPU brush cache for reusable node data - #4526

Open
YohYamasaki wants to merge 2 commits into
GraphiteEditor:masterfrom
YohYamasaki:extract-brush-cache
Open

Generalize the GPU brush cache for reusable node data#4526
YohYamasaki wants to merge 2 commits into
GraphiteEditor:masterfrom
YohYamasaki:extract-brush-cache

Conversation

@YohYamasaki

Copy link
Copy Markdown
Contributor

This extracts BrushCache from #4468 into a reusable generic cache, which will also be used in the mesh gradient #4081. It now allows to have custom eviction strategy through cache policies.
Also fixed a bug-ish type mismatch path in Cache::take() that could trigger duplicate eviction policy updates by removing and reinserting the entry.

@YohYamasaki
YohYamasaki marked this pull request as draft September 13, 2026 04:43
@YohYamasaki YohYamasaki reopened this Sep 13, 2026
@YohYamasaki
YohYamasaki marked this pull request as ready for review September 13, 2026 04:47

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 11 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread node-graph/libraries/graphene-cache/src/lib.rs
Comment thread node-graph/nodes/brush/src/basic_brush/mod.rs Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 8 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread node-graph/libraries/brush-types/src/lib.rs
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.

1 participant