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

Implement's Tycoon Wallet. #3

Merged

Conversation

leggettc18
Copy link

Implements the following

  • Tycoon Wallet item
  • Ability to upgrade from Giant's Wallet if Shopsanity is set to one item or greater
  • Adapts in-game UI to handle it
  • Adapts Dynamic Rupee Icon Enhancement to have a purple color for it
  • Adapts debugSaveEditor to allow upgrading to it (but only if allowed by the current rando settings.

@garrettjoecox
Copy link
Owner

Very nice 👍

@garrettjoecox garrettjoecox merged commit adb7ce7 into garrettjoecox:shopsanity Sep 4, 2022
garrettjoecox pushed a commit that referenced this pull request Sep 30, 2022
Co-authored-by: PurpleHato <linkvssangoku.jr@gmail.com>
Co-authored-by: GaryOderNichts <12049776+GaryOderNichts@users.noreply.github.com>

Co-authored-by: louist103 <35883445+louist103@users.noreply.github.com>
Co-authored-by: PurpleHato <linkvssangoku.jr@gmail.com>
Co-authored-by: GaryOderNichts <12049776+GaryOderNichts@users.noreply.github.com>
garrettjoecox pushed a commit that referenced this pull request Nov 2, 2022
Entirely replace gSaveContext.entranceIndex, and near the top of Gameplay_Init

(cherry picked from commit 819f669)
garrettjoecox pushed a commit that referenced this pull request Apr 22, 2024
* Fix hint generation bugs on develop-rando

1. Fixed Ganon Non-hint text from loading as Saria's Magic Hint.
2. Fixed Ganon Non-hint text from not getting saved correctly.
3. Fixed gossip stone hint generation from not generating any non-always hints on No Logic.

For #3, the hint distribution and placement algorithm was bailing out too early when it wasn't able
to place a hint. For No Logic, what it was doing was failing to place WOTH hints (since No Logic seeds
don't calculate WOTH candidacy), returning the amount of hints it failed to place, and then it called
the function to redistribute the hints, but did not call the function to attempt to place the remaining hints.

Additionally, it was not accounting for the fact that we shouldn't redistribute the hints into the categories we failed to
place a hint in, so it would redistribute hints right back into those categories. I changed it so that when DistributeHints
gets called after PlaceHints fails to place the hint, it checks if the distribution settings copies attribute was set to 0.
In this case, it breaks while looping for the type distribution settings, and moves on to the next category. Also, it now repeatedly
attempts to distribute and place hints until PlaceHints returns 0 (meaning it placed all of its hints successfully).

* Fixes some further seed-bleed type issues with hint generation.
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