-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Some XP efficiency improvements #2889
Conversation
…inimum tick time of 5 seconds Tasks inheriting from BaseTask should use `self._update_last_ran` and `_time_to_run` if they want to implement the time based running. The config to set a custom timer is named `run_interval`.
this works good with the `run_interval` configuration added to TransferPokemon and RecycleItem
…e/xp-improvements
@TheSavior review, please. |
It was looping on Moving to fort and Arrived at fort, could you check this ?
|
@menren your comment is not related to this PR and is a consequence of bad configuration. Please, go to the help channel in Slack to discuss about it. This is not the right place. |
I could be. However it does never loop with dev branch's code. Just wondering if it could be a side effect of some new features |
@menren what is your task list? |
I am using a configuration based on https://github.com/PokemonGoF/PokemonGo-Bot/blob/master/configs/config.json.example |
After applying your changes I got this error: Traceback (most recent call last): Any advice? |
@menren that configuration is not good and is prone to this kind of error that you're experiencing. |
@Naiters you're not applying the modifications correctly, probably. I was running the bot here for almost 1 hour and no errors. |
@douglascamata Which configuration are you using and is good ? |
@menren I use my configuration and I'm very satisfied. As I said, this is not the place to discuss these things, please go to the proper channel in Slack for this discussion. |
@douglascamata Hello and thank you for this. I'm currently using this but have something like 45k/H. So I guess it's my config file that isn't good enough. Would you be kind enough to share yours ? Thanks! |
@w0rd3r it can be config, bad luck or not so good place to bot. Tell me if you saw the expected around 50% experience improvement. My configuration isn't any fancy, I just bot in a very populated and pokestop dense town. This is not the place to discuss configs tho. |
I'm in Central Park since the beginning. Had more than 200k/hour with this #2491 Here is my config with your pull FYI : { },
} |
@w0rd3r I implemented exactly the same things. I think Niantic nerfed pokemon spawn and catch rate. Check without this PR how much you get and then check with the PR. |
Thanks, I'll see what happens. I went back to the classic pull cause I had better XP/H Than with your config unfortunately Your pull : [INFO] Ran for 0:58:46 Dev : [INFO] Ran for 2:00:18 |
@w0rd3r you have to use "run_interval": 60 and "ignore_item_count": true with this pull request and your config is very bad optimized for xp/h iam getting
without this pull request |
@Philipp59 thanks, could you share a good config then? |
Run interval is working great for me. Expirience wise it vary alot depending on the area. If you have area with many stops that are lured or not. For me there was not very big exp increase but I'm using path so that is to be expected |
self._validate_work_exists() | ||
self.logger = logging.getLogger(type(self).__name__) | ||
self.last_ran = time.time() | ||
self.run_interval = config.get('run_interval', 10) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like this should default to None
. It will always run every tick unless both the task utilizes _time_to_run
, and the user configures how frequently..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TheSavior the default value is 10 to make it work if the task does run in a time-based way. Otherwise if task is time-based and user don't provide run_interval
every time he will see an error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unless the task was written such that it would delay only if it was configured with an interval. Otherwise it would run every tick
@@ -12,39 +13,52 @@ class CatchLuredPokemon(BaseTask): | |||
def work(self): | |||
lured_pokemon = self.get_lured_pokemon() | |||
if lured_pokemon: | |||
self.catch_pokemon(lured_pokemon) | |||
for pokemon in lured_pokemon: | |||
self.catch_pokemon(pokemon) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens if your pokebag gets full while catching pokemon?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it just skips the catch, like it does already
Mmmm that's weird. Put a screenshot here please |
You have to be on the right folder So do it on the other terminal window that I see on the background. Do CTRL + C to abort, then |
@w0rd3r Now i get it! thanks bro! |
This is outdated. We have similar effects now implemented in dev branch. |
@douglascamata @w0rd3r I tried the dev branch for an hour and I didn't get even 40% of xp comparing to this branch.. I used the same config. can you share your config?
` |
@carlosbellino evolve pokemon task is very slow. Remove everything related to evolving pokemon. |
Hi all, |
@N3W4L still using this instead of the dev branch.. got more XP per hour.. but, atm I`m setting the dev branch (kind of confusing yet) to use the snipe feature, so.. I think dev its better once everthing is set |
@carlosbellino thanks for your answer, so I will wait. |
@douglascamata I still have way more HP/h with this config (switched to dev earlier to see) This All without lucky eggs. |
@w0rd3r you need a long running time to be more accurate. See that dev code visited 82 stops but only caught 29 pokemon and evolve only 8. In this branch you visit 76 stops, caught 42 pokemon and evolved 15. The number of catches and evolved makes a big difference. It means your tests aren't accurate. I've been getting around 79~80k exp/h using dev (no lucky egg) and I tested running for 8 hours nonstop. |
@douglascamata fair enough, as long as we're both happy with our respective XP/h, I think we're good 😊 |
@douglascamata are you using a different tasks?? Please, share your config.js |
@carlosbellino nothing fancy
|
@douglascamata what about xp/h with your config? |
2016-08-10 22:23:24,648 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Kr abby appeared! [CP 129] [Potential 0.98] [S/A/D 14/15/15] I get this error after changing the files according to yours |
@icecubeice this PR is deprecated. Use it by your own risk, I do not support it anymore as we got similar improvements merged in dev already. |
@ulovehash close to 90k exp/h at the place i'm using it. |
@douglascamata would u share your geo for bot? will post some stat screenshots to compare |
@ulovehash you guys really want to have everything at hand, god's sake. Can't even find a nice place to bot. Compare it at the place you usually run, but do LONG tests rans of at least 2 hours, that should be enough. Also take into account the number of pokestops and lures placed. |
@douglascamata ok, so to fix this will I just update my dev or? |
@icecubeice checkout dev again and forget this PR. |
@douglascamata Thank you for the config, almost the same im using.. |
@carlosbellino So you are all using this config with the original bot ? or did you implement anything to optimise it ? |
I'm using that config it with this branch even it is deprecated.. I got better results removing evolve_captured as douglascamata mentioned.. |
Ok so I'm understanding nothing. I was thinking this whole PR was deprecated. |
this PR it's deprecated.. I am using it on my own risk |
Guys, mb u have any thoughts about why am i getting this message OS X
} |
All this code was based on PR #2491 (thx a lot, @matheussampaio).
Run tasks in a time based way
BaseTask
now adds two helpers to every single task:_update_last_ran
and_time_to_run
. Both can be used in any task to ensure will only run from time to time and not in every single tick. Example:Then if the task config someone like this can be used to customise the time interval (in seconds):
Added option in SpinFort and MoveToFort to run even without inventory space (because RecycleItem may be run only after more time)
Exp result:
I saw a bigger than 50% experience per hour increase in my bot. Everyone should see something similar unless in a bad place to bot or with poor configuration.
2016-08-07 19:59:34,083 [ cli] [INFO] Total XP Earned: 13540 Average: 66236.29/h