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

Do not save capture rate in db #72

Closed
wants to merge 2 commits into from
Closed

Conversation

TurtIeSocks
Copy link
Contributor

Removes capture rate from the db, since it's entirely useless. Still sends in webhooks though.

Based off of #65 , merge that one first

Some of the most useless data in our databases
@Fabio1988
Copy link
Collaborator

Why useless? Not used in ReactMap? Which impact on storing that information?

@Mygod
Copy link
Contributor

Mygod commented Mar 25, 2023

These rates differ by account (catch badges). See also: WatWowMap/Chuck#29

@TurtIeSocks
Copy link
Contributor Author

ReactMap does not use this info, due to it being inaccurate for everyone.

@jfberry
Copy link
Collaborator

jfberry commented Mar 26, 2023

Will need rebasing

@TurtIeSocks
Copy link
Contributor Author

Should we also drop the columns and webhook data?

@Fabio1988
Copy link
Collaborator

As I can think of only Poracle uses it. somebody could reference it in the DTS

https://github.com/KartulUdus/PoracleJS/blob/develop/src/controllers/monster.js#L215

@Mygod
Copy link
Contributor

Mygod commented Mar 27, 2023

#65 already stopped processing the rates btw. 😝

@jfberry
Copy link
Collaborator

jfberry commented Mar 27, 2023

It's right the data parser doesn't record them; the question is whether we should include them in webhooks by calculation magic (or someone should provide a PR to poracle)

@TurtIeSocks
Copy link
Contributor Author

It's right the data parser doesn't record them; the question is whether we should include them in webhooks by calculation magic (or someone should provide a PR to poracle)

As in each user would save their gym badges to the poracle table to do calculations?

@jfberry
Copy link
Collaborator

jfberry commented Mar 27, 2023

It's right the data parser doesn't record them; the question is whether we should include them in webhooks by calculation magic (or someone should provide a PR to poracle)

As in each user would save their gym badges to the poracle table to do calculations?

I meant base rates

@Mygod
Copy link
Contributor

Mygod commented Mar 27, 2023

*Catch badges, not gym badges. :)

@ccev
Copy link
Collaborator

ccev commented Mar 27, 2023

I think it would be on the frontend to calculate base rates. it's as if we were to store pokemon types, names, movesets, etc. it's not on Golbat to save this.

@Fabio1988 Fabio1988 mentioned this pull request Mar 28, 2023
21 tasks
@Fabio1988
Copy link
Collaborator

Fabio1988 commented Mar 28, 2023

So for providing the catch rates in front-end we need at least the cpMultiplier, that's part of the pokemon proto. So we would need a possibility to gather that information for every mon

is the base cpMultiplier in gamemaster available @TurtIeSocks ?

@TurtIeSocks
Copy link
Contributor Author

So for providing the catch rates in front-end we need at least the cpMultiplier, that's part of the pokemon proto. So we would need a possibility to gather that information for every mon

is the base cpMultiplier in gamemaster available @TurtIeSocks ?

I don't believe so. But the jsons are available in many places

@Fabio1988
Copy link
Collaborator

already done in #74
can we create an issue to not forget this?

@Fabio1988 Fabio1988 closed this Mar 28, 2023
@Fabio1988 Fabio1988 deleted the remove-capture-rate branch March 28, 2023 12:28
@ccev
Copy link
Collaborator

ccev commented Mar 28, 2023

I'm not really sure why you'd need CPMs from the gamemaster. to calculate a catch rate, you need

  • the Pokemon's CPM (provided by Golbat)
  • the Pokemon's base capture rate (provided by the gamemaster, in the Mon's pokemon settings)
  • certain modifiers (balls, berries, medals, etc) (hardcoded values in the game's backend, can be found on the internet)

@Fabio1988
Copy link
Collaborator

I'm not really sure why you'd need CPMs from the gamemaster. to calculate a catch rate, you need

* the Pokemon's CPM (provided by Golbat)

* the Pokemon's base capture rate (provided by the gamemaster, in the Mon's pokemon settings)

* certain modifiers (balls, berries, medals, etc) (hardcoded values in the game's backend, can be found on the internet)

CPM=cpMultiplier ?
yeah it's in golbat, but it varies on the worker, true? if a worker achieves normal catch medal this will be different, no?
#79 we should continue discussion in that gh issue :)

@ccev
Copy link
Collaborator

ccev commented Mar 28, 2023

No, the CPM (yes, CP multiplier) is a 1:1 mapping to the level. each pokemon level has one specific CPM assigned to it. this is then used to calculate CP and catch rates

@Fabio1988
Copy link
Collaborator

Fabio1988 commented Mar 28, 2023

so yes, then golbat should store it or send in webhook... that's not done yet, we ignore that field now as we don't process capture rates anymore.

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.

5 participants