-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Improved handling for some edge cases with the maximum radius feature #364
Improved handling for some edge cases with the maximum radius feature #364
Conversation
Still keeps listing it. :( |
@qaqsd123 Does it at least give a warning saying there are no PokeStops nearby? Because it should. |
Didn't get that far On Sun, Jul 24, 2016, 3:02 AM Daylend notifications@github.com wrote:
|
Coords.ini is used to store the last location the bot was at. If limiting the distance was based on Coords.ini, it wouldn't be limiting the distance at all. It would only be able to travel to PokeStops x meters away from its current position, which is NOT what was intended. If you use the first coords found in Coords.ini, you would slowly drift away from your original position as the bot is reset, eventually having to put new coordinates in to keep the bot in the same place. This PR prevents edge cases where you somehow get away from the predefined boundaries. This usually happens when a user forgets to clear their Coords.ini file before moving to a new start location as set in UserSettings.settings. The point is to allow the user to save their position regardless of where they roam and continue where they left off, but keep them within a predefined area that doesn't change. Edit: This was a reply to a now deleted comment. |
You are right. How about renaming Coords.ini to LastCoords.ini then to avoid confusion? |
Sure. Someone just needs to put up a PR. Feel free to if you'd like. |
Also, if we are already going to walk back to the start, shouldn't we be spinning PokeStops along the way instead of just catching Pokemon? Could be a real long walk if someone was jumping across the country and didn't change both coords. |
Someone shouldn't be using this to walk across the country. This is for the cases where the bot is a minimal distance away from the radius. Otherwise they should see the message generated and restart the bot with the proper coordinates. |
Dont worry @Daylend , it seems perfect as is! Hope it gets accepted! |
A few users were confused when the program would endlessly list their highest Pokemon. This should hopefully fix that.