-
Notifications
You must be signed in to change notification settings - Fork 526
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
Spelunker: Implement New Game #3282
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few quick client improvements while I've got a few minutes. Didn't test any of the suggested changes.
return | ||
if "tags" not in args: | ||
return | ||
if "DeathLink" in args["tags"] and args["data"]["source"] != ctx.slot_info[ctx.slot].name: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This means that two players playing on the same slot won't send deathlinks to each other. If you don't care about supporting that, don't worry about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll have to think on this one.
Co-authored-by: Bryce Wilson <gyroscope15@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like you've learned a lot from the Yoshi's Island one, every comment here is fairly minor.
option_cave_3 = 2 | ||
option_cave_4 = 3 | ||
default = 0 | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would recommend adding StartInventoryPool here too.
Add StartInventoryPool to your imports from Options, then put start_inventory_from_pool: StartInventoryPool
in your options dataclass.
Oh yeah, also update CODEOWNERS and the AP README |
I’ll address all these tonight! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple more things
What is this fixing or adding?
Implements 'Spelunker' for the NES into AP.
How was this tested?
Technically hasn't been run publicly yet, but I've done personal tests to make sure everything is working, and given how long it takes for PRs to get reviewed, I'm sure if any issues come up I'll be able to fix them while reviews come on. Overall, the world is very small so I don't expect there to be too many problems, if any, and I've already caught the major ones that I've found.
If this makes graphical changes, please attach screenshots.
Sprites courtesy of Seafo
Author's Note:
tis a silly game, made a micro apworld in a 3 day timespan lol