Skip to content

OOM Fix for Vulkan - #56

Merged
hyblocker merged 1 commit into
hyblocker:developfrom
davidlongest:fix-about-scroll-crash
Aug 19, 2026
Merged

hyblocker merged 1 commit into
hyblocker:developfrom
davidlongest:fix-about-scroll-crash

Conversation

@davidlongest

Copy link
Copy Markdown
Contributor

Scrolling through the about page caused it to run out of descriptor pools. This uses the built-in descriptor pooling and just increases the size of the pool.

@hyblocker

Copy link
Copy Markdown
Owner

can you change the DescriptorPoolSize to a power of 2 such as 32 or 64? this is to accomodate for more textures being loaded (there are some plans in the future for this)

@davidlongest
davidlongest force-pushed the fix-about-scroll-crash branch from 0d5dfa2 to 8ba04ce Compare August 17, 2026 23:58
@davidlongest

Copy link
Copy Markdown
Contributor Author

@hyblocker I updated to 32 and rebased off the latest. I also tested going through each of the learn pages and made sure images continued loading.

@hyblocker

Copy link
Copy Markdown
Owner

it still sets DescriptorPoolSize to 10, was the change not pushed?

…ools. This uses the built-in descriptor pooling and just increases the size of the pool.
@davidlongest
davidlongest force-pushed the fix-about-scroll-crash branch from 8ba04ce to f8d551f Compare August 18, 2026 22:23
@davidlongest

Copy link
Copy Markdown
Contributor Author

Yeah I did a dumb and did an "add" but not a commit 🤦‍♂️

@hyblocker
hyblocker merged commit 7806d61 into hyblocker:develop Aug 19, 2026
3 checks 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