facebookresearch / habitat-challenge

Code for the habitat challenge
https://aihabitat.org
MIT License
306 stars 56 forks source link

Potential issue with shortest path or success criteria in ObjectNav #32

Closed devendrachaplot closed 4 years ago

devendrachaplot commented 4 years ago

Hi, I was evaluating the shortest path for the ObjectNav task. The second and third episode on val_mini seem to fail with the shortest path.

Looking at the path, it seems like the path is correct. The agent is navigating towards an instance of the target category, maybe there's some issue with success criteria?

mathfac commented 4 years ago

@devendrachaplot looking into this. If that the case, most probably issue is with shortest path, not metrics itself.

mathfac commented 4 years ago

@devendrachaplot, checked the episodes. For portion of the episodes shortest path will not lead to success. Success distance is 0.1m to a view point of the target object, but shortest path sometimes stops up to 0.2m. I would advice use new Shortestpath Planner instead of precalculated shortest paths:

goal_radius = config.TASK.SUCCESS.SUCCESS_DISTANCE / 2
follower = ShortestPathFollower(env.habitat_env.sim, goal_radius, False)
env.reset()
        while not env.habitat_env.episode_over:
            best_action = follower.get_next_action(
                closest_view_point.agent_state.position,
            )
            observations, reward, done, info = env.step(best_action)

With original shortest path (No success, distance_to_goal: 0.17m):

Screenshot 2020-05-01 13 29 14

With new Shortestpath Planner: (Success, distance_to_goal: 0.04m):

Screenshot 2020-05-01 13 29 05
devendrachaplot commented 4 years ago

Great, thanks a lot!

I had one quick follow-up question: "Success distance is 0.1m to a view point of the target object" -> How are these view points obtained?