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

Max IV Evolution implementation with hooks for other implementations #444

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Conversation

devinsba
Copy link

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
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
Copy link
Owner

jabbink commented Jul 26, 2016

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

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

@jabbink
Copy link
Owner

jabbink commented Jul 27, 2016

Good to know.

@ghost
Copy link

ghost commented Aug 1, 2016

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

@joligario
Copy link
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
Copy link
Author

devinsba commented Aug 1, 2016

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 [email protected] 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
.

task(release)
} else {
task(smartEvolve)
Copy link
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
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
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
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
Author

@jabbink this is no longer WIP status

@devinsba
Copy link
Author

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

@staurakis
Copy link

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.

None yet

5 participants