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

Auto Evolve#347

Closed
tonysguo wants to merge 8 commits into
jabbink:developfrom
tonysguo:develop
Closed

Auto Evolve#347
tonysguo wants to merge 8 commits into
jabbink:developfrom
tonysguo:develop

Conversation

@tonysguo

@tonysguo tonysguo commented Jul 25, 2016

Copy link
Copy Markdown

Bringing #98 back to develop. Please test and modify accordingly.

Additional recommended feature --

IF PokeBank > # (or FULL) && candies > #
USE Lucky Egg
EVOLVE for 30 Mins

val catch = CatchOneNearbyPokemon()
val release = ReleasePokemon()
val hatchEggs = HatchEggs()
val evolve = EvolvePoekmon()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Poekmon 👍


task(process)
task(hatchEggs)
task(evolve)

@tonysguo tonysguo Jul 25, 2016

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.

How do you check for a null string if auto_evolve in config is empty, so as to not run it?

Via != null or .length? Strings can't be null in kotlin right?

@jabbink jabbink Jul 25, 2016

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.

Strings can't be null. But you most likely have a String?, which is nullable

@Akkadius

Akkadius commented Jul 25, 2016

Copy link
Copy Markdown

Auto-evolve in release would be amazing.

Also, I like the minimum auto evolve before popping an egg

return
}
canEvolve.forEach {
val sorted = it.value.sortedByDescending { it.cp }

@Forty-Tw0 Forty-Tw0 Jul 26, 2016

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yea just change this: val sorted = it.value.sortedByDescending { it.cp } with this:
val sorted = if (settings.sortByIV) {
it.value.sortedByDescending { it.getIv() }
} else {
it.value.sortedByDescending { it.cp }
}
I dont know how to make a pull request so i only can write it here sorry!

@Agronis

Agronis commented Jul 26, 2016

Copy link
Copy Markdown
Contributor

May I suggest with a future adaptation to this that a few things are taken into account.

If the account is missing X from Pokedex, make this a priority in evolution chain.

IE: I have 200 Poliwag candy.. but I'm missing a Polywrath from my dex.

Currently, I'd be left with 8 Poliwhirl. Where as I need a Polywrath.

// Evolve highest IV/CP Pokemon into missing Pokedex entry.
// Uses closer evolution if IV/CP is equal to or greater than lower evolution tier.

pokedex_priority=true/false

Now, I'd receive one Poliwrath and two Poliwag (presuming Wrath came from Tier1 evolution).

Edit: There's actually a lot of filtering that needs to happen here.

If I have a high IV tier 2 - I want to evolve that before evolving 40 tier 1s but thats only if it's a Poke I'm interested in bringing up.

I can think of a bunch but I'm at work currently.. will come back.

@jabbink

jabbink commented Jul 26, 2016

Copy link
Copy Markdown
Owner

Afaik the required candies per evolution are now available in the api (PokemonMetaRegistry or sth like that)

@KyleBoyer

Copy link
Copy Markdown
Contributor

Can you make an option in the config to say 'ANY', thus is would evolve any Pokemon that have enough candies.

fixedRateTimer("BotLoop", false, 0, 5000, action = {
thread(block = {
task(keepalive)
task(evolve)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I like this moved down after task(catch). Otherwise those Pidgeys get released before we can evolve them for the XP.

@woutermb

Copy link
Copy Markdown

@jabbink is this branch going to be included in the next release? would love to see something i can use all the excess candy for, and evolving via API is way faster than manual! :)

@tonysguo

Copy link
Copy Markdown
Author

closing in favor of #444 (no csv)

@tonysguo tonysguo closed this Jul 28, 2016
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.