neuromorphic-paris / loris

python3 library to handle files from neuromorphic cameras
GNU General Public License v3.0
14 stars 3 forks source link

Lazy load aedat #10

Closed Tobias-Fischer closed 2 years ago

Tobias-Fischer commented 2 years ago

Allows using loris without aedat if a non-aedat format is to be read/written

biphasic commented 2 years ago

thank you Tobias. Out of curiosity, could you describe the issues you're having with the aedat package?

Tobias-Fischer commented 2 years ago

I am nearly exclusively using conda-forge in my research. aedat is a bit of a pain to package (because of rust), and might be hard to package for e.g. osx-arm64. A lazy load is a way out for this, as everything else should work on these platforms.

biphasic commented 2 years ago

a Mac user just reported this to me when using Tonic, which depends on loris: ImportError: dlopen(/opt/anaconda3/envs/wavesense/lib/python3.8/site-packages/aedat.cpython-38-darwin.so, 2): Symbol not found: ____chkstk_darwin Referenced from: /opt/anaconda3/envs/wavesense/lib/python3.8/site-packages/aedat.cpython-38-darwin.so (which was built for Mac OS X 11.0) Expected in: /usr/lib/libSystem.B.dylib in /opt/anaconda3/envs/wavesense/lib/python3.8/site-packages/aedat.cpython-38-darwin.so

Tobias-Fischer commented 2 years ago

I'm not sure how they installed tonic in a conda environment - they probably mixed conda install and pip install which is risky and can lead to issues as the one reported.

I'm just packaging the new loris version and enable a "native" build for loris (without depending on aedat), which should fix this: https://github.com/conda-forge/loris-feedstock/pull/1