search

tewtal / sm_practice_hack

Super Metroid Practice Hack
The Unlicense
27 stars 19 forks source link

v2.5.5 Update #161

Closed InsaneFirebat closed 1 year ago

InsaneFirebat commented 1 year ago

Pulling together the changes for the next update. Still needs the 32x32 projectile sprites pull before it goes live.

idlechild commented 1 year ago

Pulling together the changes for the next update. Still needs the 32x32 projectile sprites pull before it goes live.

@InsaneFirebat Did you have an optimization for the sprite WRAM, using bits so they all share the same word of memory? Just wondering if you want to pull in that 32x32 update and then add that optimization. Or you could bring in that optimization and I'll bring in 32x32 on top of it. Or you could do neither and I could just do that optimization since I don't think it would be much work.

Regardless, I might have more changes I want to make for that, perhaps make the sprites bigger so they are actual boxes and maybe change the color. I also suspect there's a simple optimization I could do for botwoon use case where if we've already drawn a box and the next box is in the same 32x32 square then skip the box. No need to iterate the whole list so it wouldn't be perfect but still an improvement.

Just letting you know once 32x32 gets merged I might have a bit more I want to do before it ships.

InsaneFirebat commented 1 year ago

I do have that RAM optimization ready to go. I can push that into your pull request in just a bit, and then you can merge it or keep working in that pull. It does need to be repointed to the next-update branch. I'll let you do that since you have more going on.

I have docs ready to go for both of us as well. Was just holding them so that the fresh debug symbols could be in the same commit.

InsaneFirebat commented 1 year ago

I'm thinking about adding a new Cutscene option to skip the game over screen. If enabled, it would try to load a savestate, then a preset, then fallback to game over. The Lion King community requested it for their practice hack, but I wasn't using built-in savestates at the time and didn't realize it would be nice in SM too. It could avoid the music change delay. Edit: I meant to post this in the other pull

InsaneFirebat commented 1 year ago

The "correct" value for min_four_corners_radius is 5, but I found setting it to 7 reduces the number of sprites during MB rainbow beam, so I set it to 7... but I'm not sure which is better. Feel free to change it back to 5.

I agree. It looks nicer at 5 but it's better to have ketchup beam fully visible. I'd leave it at 7.

Docs are included now. Everything seems good to me.