[PR #28] [MERGED] [storage][apps/code] Change sizes #203

New issue

Closed

opened 2026-05-06 13:16:14 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:16:14 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/UpsilonNumworks/Upsilon/pull/28
Author: @ArtichOwO
Created: 9/25/2021
Status: ✅ Merged
Merged: 9/25/2021
Merged by: @Lauryy06

Base: upsilon-dev ← Head: fix-size-error

📝 Commits (2)

9b04e2a [storage][apps/code] Change sizes
5254a64 [apps/code] Increase heap to 67kb

📊 Changes

3 files changed (+3 additions, -3 deletions)

View changed files

📝 apps/code/app.h (+1 -1)
📝 apps/code/variable_box_controller.h (+1 -1)
📝 ion/include/ion/storage.h (+1 -1)

📄 Description

k_pythonHeapSize -> from 100K to 67K^1,3
k_maxScriptNodesCount -> from 128 to 64²
k_storageSize -> from 32768 to 64K¹

1: See https://github.com/Omega-Numworks/Omega/issues/540
2: If not :

cannot move location counter backwards (from 00000000200xxxxx to 0000000020038000)

3: k_pythonHeapSize + k_storageSize was 132Kb before, but 64+64 is 128 (we lose 4Kb, not cool) so 67Kb for the heap (68 will throw an error)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/UpsilonNumworks/Upsilon/pull/28 **Author:** [@ArtichOwO](https://github.com/ArtichOwO) **Created:** 9/25/2021 **Status:** ✅ Merged **Merged:** 9/25/2021 **Merged by:** [@Lauryy06](https://github.com/Lauryy06) **Base:** `upsilon-dev` ← **Head:** `fix-size-error` --- ### 📝 Commits (2) - [`9b04e2a`](https://github.com/UpsilonNumworks/Upsilon/commit/9b04e2a76f043d2c846ce1a9587c795abb7ff118) [storage][apps/code] Change sizes - [`5254a64`](https://github.com/UpsilonNumworks/Upsilon/commit/5254a6418c9deeda497b0cbfa99155f519121421) [apps/code] Increase heap to 67kb ### 📊 Changes **3 files changed** (+3 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `apps/code/app.h` (+1 -1) 📝 `apps/code/variable_box_controller.h` (+1 -1) 📝 `ion/include/ion/storage.h` (+1 -1) </details> ### 📄 Description - `k_pythonHeapSize` -> from 100K to 67K1,3 - `k_maxScriptNodesCount` -> from 128 to 642 - `k_storageSize` -> from 32768 to 64K1 1: See https://github.com/Omega-Numworks/Omega/issues/540 2: If not : ```cpp cannot move location counter backwards (from 00000000200xxxxx to 0000000020038000) ``` 3: `k_pythonHeapSize` + `k_storageSize` was 132Kb before, but 64+64 is 128 (we lose 4Kb, not cool) so 67Kb for the heap (68 will throw an error) --- 🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.