Feel free to ignore this request, just a suggestion.
It became clear as part of the new reward code that world setup is inside of the Filler object instantiation. This is a side-effect which means it's not obvious. Moving world setup into Randomizer.cs seems more cleaner/more obvious.
While here, doing a little bit of cleanup on the world creation loop (bifurcate on playername, rather than the entire loop, since the loop has to run at least once anyway).
Feel free to ignore this request, just a suggestion.
It became clear as part of the new reward code that world setup is inside of the Filler object instantiation. This is a side-effect which means it's not obvious. Moving world setup into Randomizer.cs seems more cleaner/more obvious.
While here, doing a little bit of cleanup on the world creation loop (bifurcate on playername, rather than the entire loop, since the loop has to run at least once anyway).