bot wont boot up : ValueError: Expecting property name

[mehdi-S]

Expected Behavior

start normally

Actual Behavior

-> # sudo python pokecli.py -cf ./configs/config.json
[06:46:21] PokemonGO Bot v1.0
Traceback (most recent call last):
  File "pokecli.py", line 250, in <module>
    main()
  File "pokecli.py", line 229, in main
    config = init_config()
  File "pokecli.py", line 57, in init_config
    load.update(json.load(data))
  File "/usr/lib/python2.7/json/__init__.py", line 290, in load
    **kw)
  File "/usr/lib/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
    obj, end = self.scan_once(s, idx)
ValueError: Expecting property name: line 22 column 7 (char 607)

Steps to Reproduce

git clone latest build on dev branch
doing initial step
edit the config.json file

Other Information

OS: Debian x64_86
Git Commit: c4c3487

[douglascamata]

I committed fix, please test.

[mehdi-S]

after git pull :
git log : 4555466

-> # sudo python pokecli.py -cf ./configs/config.json
[07:08:16] PokemonGO Bot v1.0
Traceback (most recent call last):
  File "pokecli.py", line 250, in <module>
    main()
  File "pokecli.py", line 229, in main
    config = init_config()
  File "pokecli.py", line 57, in init_config
    load.update(json.load(data))
  File "/usr/lib/python2.7/json/__init__.py", line 290, in load
    **kw)
  File "/usr/lib/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
    obj, end = self.scan_once(s, idx)
ValueError: Expecting property name: line 22 column 7 (char 607)

[hero16]

im getting the same error as above with lastest pull. just to confirm

[BScreen]

I get a different error:

$ python pokecli.py
[13:23:45] PokemonGO Bot v1.0
Traceback (most recent call last):
  File "pokecli.py", line 250, in <module>
    main()
  File "pokecli.py", line 229, in main
    config = init_config()
  File "pokecli.py", line 55, in init_config
    if os.path.isfile(config_arg):
  File "/Users/***/Desktop/PokemonGo-Bot/_dev/PokemonGo-Bot/lib/python2.7/genericpath.py", line 37, in isfile
    st = os.stat(path)
TypeError: coercing to Unicode: need string or buffer, NoneType found

[ngkogkos]

Ensure that there is no comma separator in the last json value.

[mehdi-S]

here is the last line of my json config file, just to eject this possibility

{
    ...
    "use_lucky_egg": false,
    "evolve_captured": false,
    "catch": {
      "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
      // "Rattata": { "always_catch" : true }
    },
    "release": {
      "any": {"release_below_cp": 500, "release_below_iv": 0.8, "logic": "and"},
      // "Rattata": { "always_release" : true }
    }
}

[ngkogkos]

Try to completely remove the catch and release keys, and also the leftover comma friom the "evolve_captured" key afterwards.

[douglascamata]

@mehdi-S you have a malformed son in the configuration. Check for commas

[BScreen]

I've been following this thread and after cleaning the catch and release keys I used the command posted by @mehdi-S sudo python pokecli.py -cf ./configs/config.json and now everything seems to be working fine 👍

[mehdi-S]

what you see is exactly the defaut config.json.exemple with my value, so maybe the typo is also in the default file and need to be corrected

works fine without the catch/release key to confirm

[ngkogkos]

Is the parser fine with the // comments in the JSON structure?

[mehdi-S]

@ngkogkos
trying with and without, not the issue here

[mehdi-S]

now the bot dont release anything anymore

and frequently get into

Traceback (most recent call last):
  File "pokecli.py", line 250, in <module>
    main()
  File "pokecli.py", line 241, in main
    bot.take_step()
  File "/root/PokemonGo-Bot-myprettyusername/pokemongo_bot/__init__.py", line 36, in take_step
    location = self.navigator.take_step()
  File "/root/PokemonGo-Bot-myprettyusername/pokemongo_bot/spiral_navigator.py", line 49, in take_step
    point = self.points[self.ptr]
IndexError: list index out of range

[BScreen]

The example config looks like this:

"release": {
      "any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"},
      // "Rattata": { "always_release" : true }
    }

but since the last line is commented out there is one comma too much. It should be

"release": {
      "any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"}
    }

right?

[ironbishop]

yep, not a valid JSON with that last comma

Error: Parse error on line 9:
...logic": "or"     },      // "Rattata": { "alw
---------------------^
Expecting 'STRING', got 'undefined'

[ngkogkos]

Exactly. Same for the "catch" key also

[tpmodding]

had the same issue, after deleting the catch lines it works only if i put the config in the root direct...on windows it doestn work in the configs file....and realy...do we need for only ONE file a dir?! :/

[ironbishop]

@tpmodding files could increase in future

[mehdi-S]

@tpmodding you maybe are on the master branch, where configs/ folder dont exist already ?

[tpmodding]

no i'm on dev, i have the dir but it works only when i put the file to pokecli.py and start with python pokecli.py -cf config.json

[mehdi-S]

@tpmodding dont you try to use /configs/ instead of ./configs/ or configs/ ?

[tpmodding]

it shouldn matter? if i start with "python pokecli.py" shouldnt it auto-load from configs/config.json? or am i wrong?

[gpires]

Change the last lines of the config to:

    "catch": {
      "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "and"}
    },
    "release": {
      "any": {"release_below_cp": 340, "release_below_iv": 0, "logic": "or"}
    }
}

The problem is the comma....

[mehdi-S]

for the moment, this issue is closed

Solution :

Don't forget is you want to get rid of the // "Rattata": { "always_release" : true } line in

"release": {
      "any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"},
    }

object, to also remove comma after
"any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"},
line like so
"any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"}

do the same for the catch line and you're good to go !