Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[storage][apps/code] Change sizes #28

Merged
merged 2 commits into from
Sep 25, 2021
Merged

[storage][apps/code] Change sizes #28

merged 2 commits into from
Sep 25, 2021

Conversation

ArtichOwO
Copy link

@ArtichOwO ArtichOwO commented Sep 25, 2021 •
edited
Loading

  • k_pythonHeapSize -> from 100K to 67K1,3
  • k_maxScriptNodesCount -> from 128 to 642
  • k_storageSize -> from 32768 to 64K1

1: See Omega-Numworks#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)

@ArtichOwO
[storage][apps/code] Change sizes
9b04e2a 
k_pythonHeapSize -> 64Ko
k_maxScriptNodesCount -> 64
k_storageSize -> 64Ko
@ArtichOwO
[apps/code] Increase heap to 67kb
5254a64
@Lauryy06 Lauryy06 merged commit 0b54bf1 into UpsilonNumworks:upsilon-dev Sep 25, 2021
@ArtichOwO ArtichOwO deleted the fix-size-error branch September 25, 2021 20:48
@Lauryy06 Lauryy06 mentioned this pull request Sep 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ArtichOwO @Lauryy06