Skip to content
This repository was archived by the owner on Jul 22, 2019. It is now read-only.

Max IV Evolution implementation with hooks for other implementations#444

Open
devinsba wants to merge 18 commits into
jabbink:developfrom
devinsba:evolve
Open

Max IV Evolution implementation with hooks for other implementations#444
devinsba wants to merge 18 commits into
jabbink:developfrom
devinsba:evolve

Conversation

@devinsba

Copy link
Copy Markdown

Attempting to get #347 working more cleanly with the API and a few things I wanted

val pokebagFillPercent = ctx.api.inventories.pokebank.pokemons.size.toDouble() / ctx.profile.pokemonStorage
Log.white("Pokebag ${pokebagFillPercent * 100} % full.")
val groupedPokemon = ctx.api.inventories.pokebank.pokemons.groupBy { it.pokemonId }
if (pokebagFillPercent >= 0.7) {

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This value should be a setting

@jabbink

jabbink commented Jul 26, 2016

Copy link
Copy Markdown
Owner

FYI regarding Eevee evolves

http://gamerant.com/pokemon-go-guide-eevee-evolution-flareon-vaporeon-jolteon/

tl;dr

Rename pokemon:

  • Sparky = Jolteon
  • Rainer = Vaporeon
  • Pyro = Flareon

@marcx093

Copy link
Copy Markdown

Eevee easter egg only works first time for every pokemon. For example you can't get more than 1 vaporeon with 100% accuracy.

@jabbink

jabbink commented Jul 27, 2016

Copy link
Copy Markdown
Owner

Good to know.

@ghost

ghost commented Aug 1, 2016

Copy link
Copy Markdown

Why dont you merge this already @jabbink ? What is the problem holding it back?

@joligario

Copy link
Copy Markdown
Contributor

Why dont you merge this already @jabbink ? What is the problem holding it back?

I would guess because it is a Work In Progress (WIP). Why don't you pull it down on your own, check it out, and report back if it is operational?

@devinsba

devinsba commented Aug 1, 2016

Copy link
Copy Markdown
Author

I had been running it until the API lockdown. I have some code I haven't
pushed up and it isn't configurable at all for and end user.

On Sun, Jul 31, 2016 at 8:51 PM, JJ notifications@github.com wrote:

Why dont you merge this already @jabbink https://github.com/jabbink ?
What is the problem holding it back?

I would guess because it is a Work In Progress (WIP). Why don't you pull
it down on your own, check it out, and report back if it is operational?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#444 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIiokCfzM8Ss17rF-buTfkx_br5Cgowks5qbUMrgaJpZM4JVl8k
.

if (settings.shouldAutoTransfer) {
task(release)
} else {
task(smartEvolve)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why don't allow evolving and transferring as well?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

It happened to fit what I was planning on at the time. I am considering the final touches to make this all configurable now.

@devinsba

Copy link
Copy Markdown
Author

@jabbink should I do a rebase on this to clean up the commit history?

@devinsba devinsba changed the title [WIP] Alternate Evolve impl Max IV Evolution implementation with hooks for other implementations Aug 10, 2016
*/
class Evolve : Task {

private val DEFAULT_EVOLUTION_STRATEGY = "max_iv"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The implementation in #939 should be the default implementation but I just put mine in here as a placeholder

@devinsba

Copy link
Copy Markdown
Author

@jabbink this is no longer WIP status

@devinsba

Copy link
Copy Markdown
Author

@jabbink since #939 was merged in, should I refactor this around that implementation?

@staurakis

Copy link
Copy Markdown

i think there is a logic error in evolving.....
Firstly there should be an option/setting to be able to evolve automatically and be able to keep the evolved pokemon based on iv stats. In other words to be able to say evolve only pokemons above certain IV and KEEP them!
Secondly, check the message bellow i received:
13 Αυγ 11:49:29 [default: ProfileLoop] - Going to transfer ARBOK with CP 797 and IV 93%; reason: Evolved for XP only
After the evolve, a pokemon that shouldn't be transferred was transferred. A pokemon with IV above 90%.
It wasn't in obligatory transfer or anything like that.
By the way, if i may comment this version i am testing....it is THE GREATEST without any of the known issues.....no authentication issues, no getting the map and not doing anything...perfect in everything! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants