Bugs in the simulation code

[Emrys365]

I notice that the simulation code is not compatible with the current pyrirgen.

In tencent_challenge_rirgenerator.py#L75, it calls the function pyrirgen.generateRir, but this API has been refactored to pyrirgen.rir_generator since this commit in phecda-xu/RIR-Generator

Could you update it and also double-check the scripts?

[Emrys365]

I found the reason. I installed pyrirgen by following the link recommended in https://github.com/ConferencingSpeech/ConferencingSpeech2021/blob/master/ReadMe.md#requirements.
But the simulation code is not built based on the latest version of https://github.com/phecda-xu/RIR-Generator.
So the solution is to directly use the compiled library pyrirgen.cpython-37m-x86_64-linux-gnu.so:

cd simulation
python tencent_challenge_rirgenerator.py

[shincling]

I also found it Wangyou @Emrys365 . But I directly install the pyrirgen from the https://github.com/Marvin182/rir-generator, which solved it.

[jacktong1989]

I found the reason. I installed pyrirgen by following the link recommended in https://github.com/ConferencingSpeech/ConferencingSpeech2021/blob/master/ReadMe.md#requirements.
But the simulation code is not built based on the latest version of https://github.com/phecda-xu/RIR-Generator.
So the solution is to directly use the compiled library pyrirgen.cpython-37m-x86_64-linux-gnu.so:

cd simulation
python tencent_challenge_rirgenerator.py

May I ask how to use the compiled library "pyrirgen.cpython-37m-x86_64-linux-gnu.so"? Thanks in advance.

[Emrys365]

May I ask how to use the compiled library "pyrirgen.cpython-37m-x86_64-linux-gnu.so"? Thanks in advance.

Sorry for my late reply. I remember it was simply:

Just add the directory of pyrirgen.cpython-37m-x86_64-linux-gnu.so to $PATH

export LD_LIBRARY_PATH="/path/to/ConferencingSpeech2021/simulation":$LD_LIBRARY_PATH
cd "/path/to/ConferencingSpeech2021/simulation"
python ./challenge_rirgenerator.py