Farigh / pokeclicker-automation

An automation script suite for pokeclicker.com
GNU General Public License v3.0
38 stars 21 forks source link

Focus.PokérusCure: Add support for more options #122

Open Yzar91 opened 2 years ago

Yzar91 commented 2 years ago

Optimise road for catch Pokémon with an EV<50 whereas it passed status infected to cure. When he cured, change road for leveling EV of an another Pokémon…

TODO

Farigh commented 2 years ago

Requires filtering hatchery priority by EVs, which will be implemented as part of #48

Dessastuff commented 1 year ago

Farigh I'd like to see it run dungeons until they're resistant too? is that going to work with hatchery as well?

Farigh commented 1 year ago

Implementing dungeon would not be as easy but should not be a problem, for sure.

I don't understand the link with hatchery though...

heyboots commented 1 year ago

Yeah you shouldn't need the hatchery at all for this right? The pokedex has filtering for Uninfected/Contagious/Resistant so that info must be accessible. The hatchery is only going to give results for level 100 pokemon that are ready to become egg. I guess prioritization based on Breeding efficiency may be useful, but it also may be easier to just have the script look for routes that still have Missing Resistant marked and just chug through those in order, even if it's not the optimal solution.

Farigh commented 1 year ago

The next pokeclicker version adds a pretty interesting feature in this commit

Farigh commented 1 year ago

Suggested by @jensver in this comment

add an option to skip alternate forms for Focus.PokérusCure

GasparMassiot commented 1 year ago

It seems to me that the current PokerusCure Focus doesn't take into account UltraBeasts. My understanding is that once these pokemons are unlocked, the automation will stay on an UltraBeast Route forever. It would be nice to avoid this and maybe add an option to capture ultraBeast with beastBalls if available ?

The UltraBeast status can be checked using e.g. GameConstants.UltraBeastType[pokemons[i]] != undefined.

Also the current PokerusCure Focus doesn't take roaming pokemons into consideration. Using RoamingPokemonList we could add an option to focus on these particular pokemon.

Farigh commented 1 year ago

@GasparMassiot do you have a save reproducing the problem with ultrabeasts ?

The roamers is a bit too restrictive to be viable IMO, since it would proc really few times. But why not, I might consider it at some point

GasparMassiot commented 1 year ago

UltraBeast_Block.txt

Here is a save where the UltraBeast Block happens.

I got around the issue by creating my own findNextRouteForPokerus function that basically works the same as your findBestRouteForFarmingType function but focuses on pokerus. I can share it if you are interested but the spirit is different from the Focus.PokerusCure class you created.

I also added a moveToRoamingRoute function to farm Pokerus on roaming pokemons.