search

HEXRD / hexrd

A cross-platform, open-source library for the analysis of X-ray diffraction data.
Other
56 stars 25 forks source link

Add TBB as a dependency to HEXRD #567

Closed psavery closed 11 months ago

psavery commented 11 months ago

If TBB is installed, numba will use TBB instead of OpenMP.

In my brief testing, it appears that having TBB makes some of the workflows run a little faster. It also fixes some warnings we have been seeing about OpenMP.

psavery commented 11 months ago

I confirmed that the output of hexrd fit-grains does not change whether TBB is used or OpenMP.

pep8speaks commented 11 months ago

Hello @psavery! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 18:80: E501 line too long (81 > 79 characters)