Skip to content

[gala_kinematic] non-visual observations - #838

Merged
vincentpierre merged 11 commits into
facebookresearch:gala_kinematicfrom
vincentpierre:gala_kinematic_nonvisual
Mar 30, 2022
Merged

[gala_kinematic] non-visual observations#838
vincentpierre merged 11 commits into
facebookresearch:gala_kinematicfrom
vincentpierre:gala_kinematic_nonvisual

Conversation

@vincentpierre

Copy link
Copy Markdown
Contributor

Motivation and Context

copy of eundersander#1

Adding non-visual sensors to gala_kinematic

@vincentpierre vincentpierre self-assigned this Mar 28, 2022
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Mar 28, 2022
@vincentpierre vincentpierre changed the title Gala kinematic nonvisual [gala_kinematic] non-visual observations Mar 29, 2022
NUM_UPDATES: 60
NUM_ENVIRONMENTS: 512
SENSORS: ["DEPTH_SENSOR", "RGB_SENSOR"]
# SENSORS: ["DEPTH_SENSOR", "RGB_SENSOR", "POINTGOAL_WITH_GPS_COMPASS_SENSOR"]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed since it is in habitat_baselines/config/rearrange/gala_kinematic_ddppo.yaml

SAVE_VIDEOS_INTERVAL: 500
NUM_UPDATES: 61
NUM_ENVIRONMENTS: 512
SENSORS: ["DEPTH_SENSOR", "RGB_SENSOR", "POINTGOAL_WITH_GPS_COMPASS_SENSOR", EE_POS_SENSOR]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does EE_POS_SENSOR need to be in quotes like the others?

@eundersander eundersander left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vincentpierre
vincentpierre merged commit 4894fb6 into facebookresearch:gala_kinematic Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants