Skip to content

Split seed from the settings string#18

Open
pkmnfrk wants to merge 2 commits intoaz64:masterfrom
pkmnfrk:splitSeed
Open

Split seed from the settings string#18
pkmnfrk wants to merge 2 commits intoaz64:masterfrom
pkmnfrk:splitSeed

Conversation

@pkmnfrk
Copy link
Copy Markdown

@pkmnfrk pkmnfrk commented Feb 21, 2019

I realize there are benefits to having as part of the settings string, including easier sharing and bug reporting. However, I think for most cases, it is better to be separated. This PR makes the following changes:

Benefits:

  • You can easily remember your favourite settings, and not have to worry about accidentally generating the exact same seed
  • You can easily roll multiple games with the same settings
  • More intuitive behaviour based on other randomizers (primarily looking at OOT, but I can't think of any other randomizer I have seen that does this)

…d add the Seed and Settings to the spoiler log. Also, make the spoiler log named and pathed identically to the rom

private string SpoilerLogFilename()
{
return saveROM.FileName.Replace(".z64", ".txt");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the filename should still say SpoilerLog somewhere.

{
tSeed.Text = seedRng.Next().ToString();
}
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unpleasant mixture of tabs and spaces here and elsewhere in this pull request.

ProbablyButter pushed a commit to ProbablyButter/mm-rando that referenced this pull request May 17, 2019
Fix outputted files to all be in the same user-selected directory.
ProbablyButter pushed a commit to ProbablyButter/mm-rando that referenced this pull request May 17, 2019
Fix outputted files to all be in the same user-selected directory.
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