SpiralNavigator rewrite, mode switch #939
Conversation
| @@ -62,7 +62,7 @@ def update_web_location(self, cells=[], lat=None, lng=None, alt=None): | |||
| status = map_objects.get('status', None) | |||
| cells = map_objects['map_cells'] | |||
|
|
|||
| #insert detail info about gym to fort | |||
| # insert detail info about gym to fort | |||
There was a problem hiding this comment.
It's unfortunate that you have so many changes in this file. It makes it very difficult to read the diff here and likely that it will cause someone else merge conflicts.
There was a problem hiding this comment.
In fact, this is probably why you now have merge conflicts. :-/
|
The changes to the SpiralWalker in this PR look good to me. |
This reverts commit a710a6e.
| @@ -212,6 +212,8 @@ def work_on_cell(self, cell, position): | |||
| if hack_chain > 10: | |||
| #print('need a rest') | |||
| break | |||
| if self.config.mode == "poke": | |||
There was a problem hiding this comment.
This is a reasonable fix for now, but probably not the right long term fix. We should be removing this loop entirely so it instead only acts on one fort per event loop.
There was a problem hiding this comment.
The problem is that both workers are not giving any informations back to the bot so they need to be refactored. This quickfix should work until that is being done.
|
👍 |
* Stop farming if inventory is full * Changed spiral navigator to use pgoagpi example code until we find a better solution * pep8 reformat * Revert "pep8 reformat" This reverts commit a710a6e. * Removed debug lines * Reducing log spam
Short Description:
SpiralNavigator had code issues so I replaced it with some example code from pgoapi until we find a better solution.
Also I fixed fort-searching even if inventory is full