Skip to content

Changed random keyrings logic#73

Open
AndrewLeving wants to merge 1 commit into
Roman971:Dev-Rfrom
AndrewLeving:improved-random-keyrings
Open

Changed random keyrings logic#73
AndrewLeving wants to merge 1 commit into
Roman971:Dev-Rfrom
AndrewLeving:improved-random-keyrings

Conversation

@AndrewLeving

Copy link
Copy Markdown

Changed random keyrings to roll 50/50 per dungeon similar to how random MQ dungeons are decided.
This makes it more likely for there to be a mix of keyring and non-keyring dungeons, and less likely for there to be either all or no keyrings.

@fenhl

fenhl commented Dec 14, 2022

Copy link
Copy Markdown

Why was this closed? This seems like a reasonable change to me.

@AndrewLeving AndrewLeving reopened this Dec 14, 2022
@AndrewLeving

Copy link
Copy Markdown
Author

Why was this closed? This seems like a reasonable change to me.

At first I was considering making this a separate keyring setting, but yeah, on second thought, that's probably not necessary.

fenhl pushed a commit to fenhl/OoT-Randomizer that referenced this pull request Apr 18, 2023
@fenhl

fenhl commented Jul 30, 2023

Copy link
Copy Markdown

@AndrewLeving Would you be interested in PRing this upstream at OoTRandomizer/OoT-Randomizer? I can't guarantee that it will be accepted, but I think it makes more sense there since it's not related to anything exclusive to Dev-R.

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