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

SybtaxError cath_log event #5103

Closed
ghost opened this issue Sep 2, 2016 · 9 comments
Closed

SybtaxError cath_log event #5103

ghost opened this issue Sep 2, 2016 · 9 comments

Comments

@ghost
Copy link

ghost commented Sep 2, 2016

Expected Behavior

Works

Actual Behavior

Traceback (most recent call last):
  File "pokecli.py", line 45, in <module>
    from pokemongo_bot import PokemonGoBot, TreeConfigBuilder
  File "/home/carlos/Documentos/POKEMON/PokemonGo-Bot/pokemongo_bot/__init__.py", line 625
    self.event_manager.register_event('catch_log')
       ^
SyntaxError: invalid syntax

Your FULL config.json (remove your username, password, gmapkey and any other private info)

Output when issue occurred

Steps to Reproduce

Other Information

OS: Fedora 24

Branch: dev

Git Commit: 838fea4

Python Version: 2.7

Any other relevant files/configs (eg: path files)

Short Description

Possible solution

How it would help others

@avexus
Copy link

avexus commented Sep 2, 2016

Same here. Did a pit pull, then it won't start.
it's a good thing i have an old working version still not deleted.

@tienthanhdhcn
Copy link

tienthanhdhcn commented Sep 2, 2016

It is some minor problem. You can fix it by doing the following steps:

  1. In pokemongo_bot/init.py line 623 add ")"
    self.event_manager.register_event(
    'save_spawn',
    parameters=('pokemon','location')
    )
  2. In cell_workers/pokemon_catch_worker.py remove from "<<<<<<< HEAD" to "=======" (line 666 to 699) and remove ">>>>>>> e83c7e6" ( conflict).

Edit: you need to delete line 153 self.dump_pokemon_spawn(pokemon) as well on the cell_workers/pokemon_catch_worker.py

@avexus
Copy link

avexus commented Sep 2, 2016

guess it's no testing before commit

@avexus
Copy link

avexus commented Sep 2, 2016

@tienthanhdhcn, could you change it and commit back to the dev branch? I think more people will have the same problem.

@tienthanhdhcn
Copy link

@avexus sorry I cannot have permission to commit to the project :)

@r3pek
Copy link

r3pek commented Sep 2, 2016

yep, have the same problem as of commit 838fea4

@ghost
Copy link
Author

ghost commented Sep 2, 2016

Post here You solution @tienthanhdhcn

@loflol
Copy link

loflol commented Sep 2, 2016

thanks @tienthanhdhcn
It's work fine ow

@oralunal
Copy link
Contributor

oralunal commented Sep 2, 2016

Fixed. You need to pull

@oralunal oralunal closed this as completed Sep 2, 2016
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

No branches or pull requests

5 participants