Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

incomplete scanning #34

Closed
Sigi-cz opened this issue Jul 27, 2016 · 8 comments
Closed

incomplete scanning #34

Sigi-cz opened this issue Jul 27, 2016 · 8 comments
Labels

Comments

@Sigi-cz
Copy link

Sigi-cz commented Jul 27, 2016

I set scan radius to 15000, that makes 27k locations. The scan never finishes all of them and restarts after 13minutes of scanning. You can se the unscanned area on the picture, part of log is here.
I guess that has something to do with need to finish cycle in 15mins to be 100% sure to catch all spawns?
That's why I'd love to see possibility to use more accounts for scanning.

@nborrmann
Copy link
Collaborator

This shouldn't be happening

@nborrmann nborrmann added the bug label Jul 27, 2016
@ghost
Copy link

ghost commented Jul 27, 2016

@Sigi-cz if you execute the phyton server multiple times with multiple accounts you can distribute the scanning over more accounts, they inheritly share the same DB file. we use this in the hosted solution https://pkmn_wrn.itdata.at

One for vienna and one for Wiener Neustadt (austria both) ATM you only see gyms as the PTC login server is down :(

Also i adapted the map for hosted solution with notifications and notification areas instead of moving the scan locations center

@Sigi-cz
Copy link
Author

Sigi-cz commented Jul 27, 2016

it's a bit OT, but I'll ask, If those two instances overlap, will they create duplicities in DB? I guess not.

@ghost
Copy link

ghost commented Jul 27, 2016

havent tried yet as we have 50km apart.

@favll
Copy link
Owner

favll commented Jul 27, 2016

No, it will not create duplicates. Pokemon are stored by their encounter id which is unique in the db. 👍

@claimer21
Copy link

@MarkusFlorian Do you know of a way to allow the multiple accounts to use separate Gmaps API keys?

@ghost
Copy link

ghost commented Jul 28, 2016

@moggd It's quite easy to adapt the gmaps key to be part of the script call arguments instead of the init config but this would need separate adaption. The code currently does not provide such a functionality.

@favll
Copy link
Owner

favll commented Aug 8, 2016

Closing due to the extensive rewrite.

@favll favll closed this as completed Aug 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants