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.

Sign up for GitHub

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

Jump to bottom

Randomizer Class and Seed Initialization #9

Closed
AECX opened this issue Oct 7, 2021 · 1 comment
Closed

Randomizer Class and Seed Initialization #9

AECX opened this issue Oct 7, 2021 · 1 comment
Assignees
@AECX
Labels
architecture Architectural changes or suggestions

Comments

@AECX
Copy link
Member

AECX commented Oct 7, 2021

There should be more happening in (or through) the custructor of the Randomizer class, initializing the seed through the load event was a workaround that should be resolved;
Things should happen in their respective class if possible.
@AECX AECX added the architecture Architectural changes or suggestions label Oct 7, 2021
@AECX AECX self-assigned this Oct 7, 2021
jdflyer pushed a commit to jdflyer/Randomizer that referenced this issue Jun 30, 2022
@kipcode66
fix: removed unwanted (s)bss symbols (zsrtp#9)
60add7b
@lunarsoap5
Copy link
Contributor

Closing as no longer relevant with current Randomizer architecture

@lunarsoap5 lunarsoap5 closed this as not planned Won't fix, can't repro, duplicate, stale Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AECX AECX

Labels
architecture Architectural changes or suggestions
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AECX @lunarsoap5