search

VIRL-Platform / VIRL

(ECCV 2024) Code for V-IRL: Grounding Virtual Intelligence in Real Life
https://virl-platform.github.io/
299 stars 10 forks source link

Some errors when runing the code #3

Open bf-yang opened 2 months ago

bf-yang commented 2 months ago

Thanks for your great job. I meet the following three problems when running the code.

  1. When I run the "python launcher.py --cfg_file cfgs/peng/peng.yaml", is usually reports: "ValueError: Cannot find geocode for NYU Card Center address". But sometimes it works, but it still reports an error: "SyntaxError: EOL while scanning string literal". image

  2. When I modify the WAY_POINTS_LANGUAGE to other descriptions, it may report an error: image

  3. I still don't know how to set the WAY_POINT_PARSE_TEMPLATE. Should it be set as the default value like this "WAY_POINT_PARSE_TEMPLATE: NAVIGATION_LANGUAGE_TO_WAYPOINT_LIST"?

Could you please help me? Thanks~

jihanyang commented 2 months ago
  1. This is a problem caused by the update of gpt model (refer to https://github.com/VIRL-Platform/VIRL/blob/main/docs/FQA.md). You can fix this by just changing the gpt-4-1106-preview to gpt-4-turbo: https://github.com/VIRL-Platform/VIRL/blob/237b7e110e24be1b32fabc4977f8fd35210f8e6d/tools/cfgs/peng/peng.yaml#L31

I will update this default config.

  1. Can you show your modification?
  2. You don't need to set WAY_POINT_PARSE_TEMPLATE, it's our designed prompt template to extract waypoints from free-form human language: https://github.com/VIRL-Platform/VIRL/blob/237b7e110e24be1b32fabc4977f8fd35210f8e6d/virl/lm/prompt/places_templates.py#L169-L178