StingraySoftware / stingray

Anything can happen in the next half hour (including spectral timing made easy)!
https://stingray.science/stingray
MIT License
174 stars 144 forks source link

Recommended package `tbb` is only for x86 #839

Closed taldcroft closed 2 months ago

taldcroft commented 2 months ago

Description of the Issue

The installation instructions recommend installing tbb. When I tried this on my Mac Silicon, pip complained that the package does not exist. This caused some confusion until I discovered that this package is provided by Intel for x86 processors.

Suggestions regarding the Issue

Update the docs to mention that tbb is only for x86 processors.

matteobachetti commented 2 months ago

Thanks @taldcroft for reporting the issue!

matteobachetti commented 2 months ago

Removing tbb from dependencies altogether seems to resolve the issue. I think that numba is now taking care of installing it when needed. We had probably added it at a time when this did not happen automatically.