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.

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

Rocci bird and Memory fixes #14

Merged
merged 52 commits into from
Jan 16, 2024
Merged

Rocci bird and Memory fixes #14

merged 52 commits into from
Jan 16, 2024

Conversation

bhansconnect
Copy link
Collaborator

This adds rocci bird.

It also deals with a major memory bug in the platform.


Overall, I am happy with it. Though currently it generates really big cartridges that often don't fit in the 64KB limit. Rocci bird for example:

  1. Now regenerates ever sprite every frame instead of storing them in the model to help avoid closure capture code explosion. (probably compiler bug)
  2. The sprites actually totally reallocate and fill out list every frame a byte a time cause constants don't work correctly. (compiler bug)
  3. It must be built with a lower memory size to fit. I use -Dmem-size=8192. (fact of life with tight sizing)
  4. Even with that lower memory constraint, it still is too big until I run wasm-opt -Os at which point it is exactly 64KB. (useful extra tool)

Luckly for debug and development, this all just works (it ignores memory constraints). Also, perf seems relatively unimportant in wasm4 due to how small all the data is.

@lukewilliamboswell lukewilliamboswell merged commit 43e88cd into main Jan 16, 2024
1 check passed
@lukewilliamboswell lukewilliamboswell deleted the rocci-bird-mem branch January 16, 2024 21:30
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