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

Sniper fixes #5962

Merged
merged 2 commits into from
Mar 14, 2017
Merged

Sniper fixes #5962

merged 2 commits into from
Mar 14, 2017

Conversation

davidakachaos
Copy link
Contributor

Short Description:

Fixes for these issues I noticed:

  • Fixes an issue with a source hiding latitude and longitude of Pokemons when using it without a subscription.
  • Also converts IV to float instead of an integer. Some sites use IVs like 96.7
  • Adds a cooldown to sniping, to prevent Niantic from seeing you jumping all over the world 😄
  • Fixes an logic issue, where sniper would snipe Pokemon below the special IV instead of that and above.

selfie-0
GitHub Selfies

Fixes an issue with a source hiding latitude and longitude of Pokemons
when using it without a subscription.
Also converts IV to float instead of an integer. Some sites use IVs like
96.7
@mention-bot
Copy link

@davidakachaos, thanks for your PR! By analyzing the history of the files in this pull request, we identified @YvesHenri, @Gobberwart and @MerlionRock to be potential reviewers.

@MerlionRock
Copy link
Contributor

Wondering how does cool down prevent Niantic not seeing you jumping around the world.... Lol....

@davidakachaos
Copy link
Contributor Author

davidakachaos commented Mar 14, 2017

It was an idea I had; after sniping Niantic might "flag" an account to be more closely monitored. Nothing based on proof, just feeling like something I would do when I was combatting bots...
And no, it wouldn't prevent it, but it does screw with a detecting strat to see if some accounts are sniping...
I would be happy to remove the cooldown if you think it has no use at all 😉

@pogarek
Copy link
Contributor

pogarek commented Mar 14, 2017

I think it would be good to expose cooldown to config.json , with default behavior like it was before the PR. :) Plus, of course, a proper modification to sample config and docs :)

@davidakachaos
Copy link
Contributor Author

@pogarek Done 😉

@pogarek
Copy link
Contributor

pogarek commented Mar 14, 2017

👍
let's test it out on dev :) However I'll stick with my current config. Works for a last few months :-P

Approved with PullApprove

@pogarek pogarek merged commit fca25e5 into PokemonGoF:dev Mar 14, 2017
@pogarek
Copy link
Contributor

pogarek commented Mar 14, 2017

merged #5962

@davidakachaos davidakachaos deleted the sniper_adjustments branch March 14, 2017 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants