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

Make a universal switchyard for GPS assignment. Add config for altitude. #2986

Closed
wants to merge 6 commits into from
Closed

Make a universal switchyard for GPS assignment. Add config for altitude. #2986

wants to merge 6 commits into from

Conversation

BoboFraggins
Copy link

@BoboFraggins BoboFraggins commented Aug 8, 2016

Short Description:

Pull all calls to the assignment of the GPS coordinates into a single object. For now this allows for assigning the initial altitude to use instead of always claiming exactly 'sea level'. Longer term this will allow for simulating decaying GPS accuracy (e.g. - +/- 50m when the bot launches and dropping down to +/- 10m as it is running). Those sort of variables would also be configured in the 'gps' section of the config file.

Assumptions:

  • Nowhere can I find any reference to the units that altitude should be in. I assume meters.
  • I made a 'sensors' folder for these types of objects in anticipation of other sensors like the accelerometer.

Closes #2762

@MZorzy
Copy link

MZorzy commented Aug 8, 2016

on pokemongo_bot/cell_workers/follow_spiral.py line 93
'current_position': (point['lat'], point['lng'], 0),

@BoboFraggins
Copy link
Author

@MZorzy Yep, sorry missed that one.

@elicwhite
Copy link
Contributor

@BoboFraggins It looks like you have merge conflicts.

@douglascamata Can you review this please?

…_assignment

Conflicts:
	pokecli.py
	pokemongo_bot/cell_workers/catch_visible_pokemon.py
	pokemongo_bot/cell_workers/move_to_map_pokemon.py
	pokemongo_bot/step_walker.py
@BoboFraggins
Copy link
Author

Conflicts resolved.

@elicwhite
Copy link
Contributor

@BoboFraggins, thanks

@douglascamata what are your thoughts here?

@douglascamata
Copy link
Member

@TheSavior thoughts tomorrow, sorry... too late.

@douglascamata
Copy link
Member

@BoboFraggins I'm thinking about starting a lib, api_wrapper or services folder where we will put all the classes the wraps API logic, like inventory.py and other stuff. You mind trying to start this already? What are you thoughts about this structure proposal?

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