facebookresearch / sound-spaces

A first-of-its-kind acoustic simulation platform for audio-visual embodied AI research. It supports training and evaluating multiple tasks and applications.
https://soundspaces.org
Creative Commons Attribution 4.0 International
319 stars 55 forks source link

Bug with quick tutorial code in soundspaces 2.0 #139

Open hiro-oda opened 3 months ago

hiro-oda commented 3 months ago

Hi

It seems that there's an error stating that habitat_sim.sensor.RLRAudioPropagationChannelLayoutType is a Nonetype object in line 36 of the file soundspaces2_quick_tutorial.ipynb, where audio_sensor_spec.channelLayout.type = habitat_sim.sensor.RLRAudioPropagationChannelLayoutType.Binaural is specified.

Indeed, according to the official documentation of habitat-sim (https://aihabitat.org/docs/habitat-sim/habitat_sim.sensor.html), habitat_sim.sensor.RLRAudioPropagationChannelLayoutType is set to None. Could this be a mistake in the tutorial code?

WYXG233 commented 2 weeks ago

@hiro-oda Hi, I have encountered the same problem. Have you been able to find out how to update it?

hiro-oda commented 2 weeks ago

@WYXG233 Hi. I haven't fixed this problem yet, but I think this bug is caused by a version difference in habitat.sim. So, it might be a good idea to downgrade the version.

ChanganVR commented 1 week ago

@hiro-oda @WYXG233 the code runs fine for me. Did you follow the installation document to install Habitat-Sim properly? Also please refer to the following document for SS 2.0 APIs.

https://github.com/facebookresearch/sound-spaces/blob/main/SoundSpaces2.md

WYXG233 commented 1 week ago

@ChanganVR, thanks for your response. Could you provide the version of Habitat-Sim you are currently using? I used the habitat-sim-0.3.1, which will output:

In:
  audio_sensor_spec.channelLayout.type = habitat_sim.sensor.RLRAudioPropagationChannelLayoutType.Binaural

AttributeError: 'NoneType' object has no attribute 'Binaural'