ebu / ebu_adm_renderer

The EBU ADM Renderer, written in Python, is the reference implementation of EBU Tech 3388
https://ear.readthedocs.io
BSD 3-Clause Clear License
79 stars 13 forks source link

EAR crashes rendring cartesian ADM.... #41

Closed WernerBleisteiner closed 2 years ago

WernerBleisteiner commented 2 years ago

...with message

'ObjectCartesianPosition' object has no attribute 'distance'

which seems odd, as cartesian should not have a distance parameter.....

respective axml attached TAT prinzip Hoffnung ATMOS ADM neuer start.wav--amxl_DFMfix.xml.zip

Any workaround possible?

tomjnixon commented 2 years ago

Hi, thanks for reporting this; unfortunately I couldn't get this error to occur. Please could you let me know:

WernerBleisteiner commented 2 years ago

Thanks Tom. Just succeeded this time.

Sorry, I may have got confused, as my aim is to create a binaural version from this file using https://github.com/IRT-Open-Source/binaural_nga_renderer As this one's based on EAR, I didn't quite get if installing nga-binaural would effect (corrupt) a previously installed ear. If I switch to git, I think I'd better go for pipenv...

As nga-binaural failed, I thought of cross check with EAR (and may had gotten things wrong). Was astonished, as I could import this ADM in EAR Production Suite (however had to use a 0.7.0 beta first, as it crashed on export).

BASH-LOG-debug-ear-render-Dolby-ADM-TATORT-succes.rtf.zip CLOSED

tomjnixon commented 2 years ago

Great, I've opened a couple of pull requests to make binaural_nga_renderer compatible with the latest EAR master, and to (hopefully) fix your bug.

I would recommend using the latest EAR; you could use two virtualenvs to keep them separate. There are some instructions for using virtualenvs, and installing the latest version, here: https://ear.readthedocs.io/en/latest/install.html#use-a-virtualenv

WernerBleisteiner commented 2 years ago

Thanks. Great that you've invested energy into readthedocs. That'll help me a lot.