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

improved stepper that walks using Google Directions API #721

Merged
merged 2 commits into from
Jul 25, 2016
Merged

improved stepper that walks using Google Directions API #721

merged 2 commits into from
Jul 25, 2016

Conversation

th3w4y
Copy link
Contributor

@th3w4y th3w4y commented Jul 24, 2016

Short Description:
In order to try it, add below lines to pokecli.py

  • from pokemongo_bot.polyline_stepper import PolylineStepper
  • bot.stepper=PolylineStepper(bot) [after the bot.start()]

Fixes:

  • fixed percentage_walked calculation
  • fixed some bugs in polyline_stepper
  • changed speed to 100m/s inside the test
  • merged with current (dev) - some conflicts solved

@elicwhite
Copy link
Contributor

This is great work.

It would make things much easier if your PR actually described the problem you are trying to solve and your changes. Having many PRs that just say Polyline isn't very helpful to reviewers

@th3w4y th3w4y changed the title Polyline improved stepper that walks using Google Directions API Jul 24, 2016
@jtdroste
Copy link
Contributor

Cloud you rebase your commits, and merge them into one commit?

@th3w4y
Copy link
Contributor Author

th3w4y commented Jul 25, 2016

@jtdroste Thanks for the suggestion - i just learned how to do it now

proposed_origin = polyline_walker.points[0]
proposed_destination = polyline_walker.points[-1]
proposed_lat = proposed_origin[0]
proposed_lng = proposed_origin[1]
if proposed_lat != lat and proposed_lng != lng:
logger.log('[#] Using _old_walk_to to go to the proposed_origin: ' +str(proposed_origin))
Copy link

@ProjectBarks ProjectBarks Jul 25, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use .format for args

@th3w4y th3w4y closed this Jul 25, 2016
@th3w4y th3w4y reopened this Jul 25, 2016
@solderzzc
Copy link
Contributor

Test code, not called, so merged.

@solderzzc solderzzc merged commit 149a357 into PokemonGoF:dev Jul 25, 2016
MFizz pushed a commit to MFizz/PokemonGo-Bot that referenced this pull request Jul 29, 2016
* improved stepper that walks using Google Directions API

* improved logging
hoanghuynh pushed a commit to hoanghuynh/PokemonGo-Bot that referenced this pull request Sep 28, 2016
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.

5 participants