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

[QoL] Allow Own Tempo Rockruff to spawn in the wild. #2782

Closed
wants to merge 11 commits into from

Conversation

Smewkie
Copy link
Contributor

@Smewkie Smewkie commented Jul 4, 2024

What are the changes?

Adds Rockruff to battle-scene.ts in the section for wild pokemon form spawns, assume it's an oversight that it's not there at all due to Own Tempo's original implementation(it didn't work and wasn't a form previously).

This will give any Rockruff that spawns a 50/50 chance to have Own Tempo form, I don't see this as being much of a buff, as Froakie and it's line have the same logic for Battle Bond, which is arguably a much better ability and pokemon combo.

Arguably, for "balance", this could be changed to be a 1/3 chance instead, if the balance team would rather have that, that's easily doable I think?

Why am I doing these changes?

Currently, own tempo form was locked to finding a lycanroc that had the form, which is a little obtuse, since a dusk form lycanroc can currently spawn without having the form.

What did change?

Added one line to battle-scene.ts, line 1225.

Screenshots/Videos

image

How to test the changes?

Set your enemy's pokemon species in overrides to be rockruff
catch some rockruffs
some of them have own tempo! yay!

Checklist

  • [Y] There is no overlap with another PR?
  • [Y] The PR is self-contained and cannot be split into smaller PRs?
  • [Y] Have I provided a clear explanation of the changes?
  • [Y] Have I tested the changes (manually)?
    • Are all unit tests still passing? (npm run test)
  • [Y] Are the changes visual?
    • [Y] Have I provided screenshots/videos of the changes?

@flx-sta flx-sta added (Legacy) Bug Legacy Label, don't apply to new issues/PRs Enhancement New feature or request and removed Enhancement New feature or request labels Jul 4, 2024
@f-fsantos f-fsantos changed the base branch from main to beta July 10, 2024 17:46
@killeroid356
Copy link

Just asking if this is planned to be merged anytime soon? really want to use dusk lycanroc :)

@Smewkie
Copy link
Contributor Author

Smewkie commented Jul 30, 2024

Just asking if this is planned to be merged anytime soon? really want to use dusk lycanroc :)

there were concerns from the dev team that this would shoehorn too many wild rockruffs into being dusk form, so I need to learn how to lower the chance of own tempo form

swap dusk form lycanroc to be formindex 1, so that when you catch a dusk lycanroc you get own tempo rockruff
Removes Lycanroc's forms from being a time of day based when spawning in the wild(and in trainers parties!), and instead make it based on a % chance for each form, with each form being the same BST I think it hardly matters which one you find personally.

Also sets Own Tempo Rockruff to be a 20% chance to spawn when a Rockruff is generated.
@Smewkie Smewkie closed this Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(Legacy) Bug Legacy Label, don't apply to new issues/PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants