Release memory on a few occasions #694
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds explicit memory releases at strategic points to free up memory earlier in long-running data processing operations. The changes use Rust's drop() function to release large data structures as soon as they are no longer needed, rather than waiting for them to go out of scope naturally.
- Releases
height_gridimmediately after applying Gaussian blur in elevation processing - Releases
elementsandhighway_connectivityafter terrain element processing completes
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/elevation_data.rs | Drops height_grid after blur operation to free memory before Minecraft height conversion |
| src/data_processing.rs | Drops elements and highway_connectivity after element processing loop to free memory before ground generation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
⏱️ Benchmark run finished in 0m 31s 📈 Compared against baseline: 30s 🟢 Generation time is unchanged. 📅 Last benchmark: 2026-01-06 16:04:23 UTC You can retrigger the benchmark by commenting |
No description provided.