Closed vinnymac closed 8 years ago
Im not sure which is your bit and which is the test result from the site?
@Wrexial both test results are from the pull request methods. I will display the ones from the site below. I am certain they would be different right now. I am pretty sure the blossom code considers existing powerups num_upgrades
, but both of these pokemon have 0.
These are the methods I ran. First param is the trainer level, second is num_upgrades
, third is the pokemon's cp_multiplier
utils.getMaxCandyCostsForPowerup(20, 0, 0.5974000096321106)
utils.getMaxStardustCostsForPowerup(20, 0, 0.5974000096321106)
utils.getMaxCandyCostsForPowerup(20, 0, 0.5507926940917969)
utils.getMaxStardustCostsForPowerup(20, 0, 0.5507926940917969)
10 candies | 11,000 stardust
11 candies | 11,500 stardust
22 candies | 24,800 stardust
23 candies | 25,600 stardust
@Wrexial gave me this sheet https://docs.google.com/spreadsheets/d/118FlgG9YMrU6Ae74cwuhl_AXyEv0QEw9A7Pfvo8MJYI/edit#gid=0
I updated the blossom methods with it and hopefully that will make the costs more accurate now.
@Wrexial and @mackhankins let me know what you think of this so far. It is doing what I wanted it to, but I don't think I am getting the right costs yet, which may have something to do with the functions from Blossom. If you look at http://pokemongo.gamepress.gg/power-up-costs It seems a bit off from those numbers.
I am sure that iterating with
numOfUpgrades
works properly. I just think that calculated costs for stardust and candies is off somehow.Some Test Results
Charmander
Bulbasaur
If you look at the first result, you can see the candies go from 2 to 4 to 7 to 10 (2 + 2 + 3 + 3). From levels 20.5-22. So in just 4 levels it calculates 10 candies. But based on the power up costs site from 20.5 to 22 it should be 2 to 5 to 8 to 11 (2 + 3 + 3 + 3). I am not sure who is right or wrong here since we are pinning Blossoms code vs the Power Up Costs from gamepress. Let me know your thoughts.