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.

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

Flip bit to make not shiny instead of rerolling #3640

Closed
wants to merge 1 commit into from

Conversation

AreaZR
Copy link

@AreaZR AreaZR commented Dec 7, 2023

Description

I don't have anything for the inverse yet. I just think this is better than a while loop.

Better than advancing the RNG when it has a shiny and allows for more IVs to be possible or accessible.

Discord contact info

roseareazero

@mrgriffin
Copy link
Collaborator

Some context for whoever reviews this: because #3438 is able to cheat it also avoids looping.

I don't think there's anything wrong with this approach though. "So many times" is probably an overstatement in a lot of cases, naively we'd think it's 1/4096 to roll the RNG twice, 1/(4096*4096) to roll the RNG thrice, etc.

Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo left a comment

Choose a reason for hiding this comment

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

Use the PR template on the description.

@AreaZR
Copy link
Author

AreaZR commented Dec 8, 2023

Use the PR template on the description.

Okay, I edited the PR description and everything.

This is better than advancing the RNG because it avoids looping and it makes more IVs accessible.
@Bassoonian
Copy link
Collaborator

Irrelevant now that #3438 is a thing

@Bassoonian Bassoonian closed this Dec 27, 2023
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.

4 participants