Skip to content

Fix Vulkan headless backend resize#4030

Merged
adrian-cojocaru merged 3 commits intomaplibre:mainfrom
adrian-cojocaru:headless-resize
Jan 8, 2026
Merged

Fix Vulkan headless backend resize#4030
adrian-cojocaru merged 3 commits intomaplibre:mainfrom
adrian-cojocaru:headless-resize

Conversation

@adrian-cojocaru
Copy link
Collaborator

Similar to #4018 this forces the read texture to resize and match the renderable resource.
Minor rendering stats memory fixes exposed by Texture2D::setSize.

@github-actions github-actions bot added core Changes that affect the C++ core of MapLibre Native Vulkan labels Jan 6, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

Bloaty Results 🐋

Compared to main

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  [ = ]       0  [ = ]       0    TOTAL

Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-4030-compared-to-main.txt

Compared to d387090 (legacy)

    FILE SIZE        VM SIZE    
 --------------  -------------- 
   +47% +55.0Mi  +451% +26.9Mi    TOTAL

Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-4030-compared-to-legacy.txt

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

Bloaty Results (iOS) 🐋

Compared to main

    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +0.0%    +160  [ = ]       0    TOTAL

Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results-ios/pr-4030-compared-to-main.txt

@adrian-cojocaru adrian-cojocaru merged commit 32eed7a into maplibre:main Jan 8, 2026
46 checks passed
@adrian-cojocaru adrian-cojocaru deleted the headless-resize branch January 8, 2026 10:55
@CommanderStorm
Copy link
Member

@louwers might it be possible for you to do a core release? I think this would help maplibre/maplibre-native-rs#115

@Murmele
Copy link
Contributor

Murmele commented Jan 12, 2026

@louwers might it be possible for you to do a core release? I think this would help maplibre/maplibre-native-rs#115

Would be nice, so I don't have to compile my self

@louwers
Copy link
Member

louwers commented Jan 12, 2026

Yes will do, let me get one more fix in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Changes that affect the C++ core of MapLibre Native Vulkan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants