Skip to content

Updated rand for VMan_2002 presents#130

Open
VMan-2002 wants to merge 6 commits intoBalatro-Potato-Patch:mainfrom
VMan-2002:vman_new_rand
Open

Updated rand for VMan_2002 presents#130
VMan-2002 wants to merge 6 commits intoBalatro-Potato-Patch:mainfrom
VMan-2002:vman_new_rand

Conversation

@VMan-2002
Copy link
Copy Markdown

@VMan-2002 VMan-2002 commented Dec 30, 2025

no more math.random !!!!!!!!!!!!!

@BakersDozenBagels
Copy link
Copy Markdown
Contributor

Why do you need vman.randseed? All it seems to do is make the random values intentionally non-deterministic1, which seems counter to the point of removing math.random calls. And for future reference, pseudoseed (and anything else calling it) already do the same thing you're trying to do here, meaning you don't need to do it yourself.

Footnotes

  1. Specifically, since the index here is saved in G rather than G.GAME, playing a hand and then save-scumming will result in different RNG rolls.

@VMan-2002
Copy link
Copy Markdown
Author

since the index here is saved in G rather than G.GAME, playing a hand and then save-scumming will result in different RNG rolls

oops my mistake

it's properly deterministic now.
oopsy!
@VMan-2002
Copy link
Copy Markdown
Author

is fixed now

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