Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[In progress] Improve efficiency of L30 accounts #2336

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

Conversation

AEtHeLsYn
Copy link
Contributor

@AEtHeLsYn AEtHeLsYn commented Oct 10, 2017

Description

[In progress] This PR actually increases the usage of L30 accounts.

With this PR we aim to improve efficiency of level 30 accounts by using the same account each parse loop instead of using several accounts if more than one encounter is needed. This is particularly helpful when scanning nests, as there may be some whitlisted pokemon together and we may need to use several level 30 accounts to encounter all pokemons before scanning the next step.

Motivation and Context

The change potentially reduces the usage of level 30 accounts.

How Has This Been Tested?

Local testing map

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@AEtHeLsYn AEtHeLsYn changed the title Improve efficiency of L30 accounts [In progress] Improve efficiency of L30 accounts Oct 12, 2017
@sebastienvercammen sebastienvercammen added this to the 4.1.1 milestone Oct 17, 2017
@javisabalete
Copy link
Contributor

What about this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants