Skip to content
This repository has been archived by the owner on Aug 28, 2021. It is now read-only.

confusion about habitat-api and habitat-sim version #7

Closed
StOnEGiggity opened this issue Oct 6, 2019 · 4 comments
Closed

confusion about habitat-api and habitat-sim version #7

StOnEGiggity opened this issue Oct 6, 2019 · 4 comments

Comments

@StOnEGiggity
Copy link

Hi,

I found we can't reproduce the experimental results after updating habitat-sim and habitat-api. I know it may be unrelated to this repo. But I am confused about the reason.
I hope that I can get some advice from you. It's useless even though I check their commit log.

Thanks

@danmou
Copy link

danmou commented Oct 29, 2019

I don't know if this is the cause, but the latest version of Habitat changed the default config so the episode order is now shuffled by default (which makes more sense, but might give different results). You can try with the old behavior by adding this to the config:

ENVIRONMENT:
  ITERATOR_OPTIONS:
    SHUFFLE: False

@StOnEGiggity
Copy link
Author

Sorry for replying late. I also find the pre-trained models perform worse after I use new habit-api and habitat-sim. One issue in habitat-api told that it happened because of different order of action space. However, this method includes three actions. I believe there is no influence for this repo. I will check again. Thanks a lot.

@StOnEGiggity
Copy link
Author

The problem is related to facebookresearch/habitat-lab#148. Following the instruction, I change the config with POINTGOAL_WITH_GPS_COMPASS_SENSOR . The results look good.

I will close this issue. Thanks.

@phongnhhn92
Copy link

@StOnEGiggity
Hi, I am on the master branch of habitat-sim and habitat-api, how can i find the config file to apply above changes ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants