Zack

Cleaned up some of the logic in `warp_image` directly. Expected speedup ~4-5%

Eigen based anglesToGvec implementation

1

Convert the C implementation of anglesToGvec to C++, using the Eigen library for optimized matrix performance. Expected single-thread performance increase: 10% Future improvements can be made by building and optimizing...

Lint tomoutil and nfutil

2

Mostly this cleans up the `nfutil.py` script. I linted the code, simplified some logic, and cut out some unnecessary commented out code and arguments. This will make it a bit...

The [`PlaneData`](https://github.com/HEXRD/hexrd/blob/b38b4c94ee0343f975e8fb51e69ad7d2314e0bd7/hexrd/material/crystallography.py#L483) class should have unit tests which thoroughly cover the use cases of its methods. @psavery to advise.

The [Crystallography `PlaneData`](https://github.com/HEXRD/hexrd/blob/b38b4c94ee0343f975e8fb51e69ad7d2314e0bd7/hexrd/material/crystallography.py#L483) class must have all public methods documented with docstrings. All associated functions should also have proper type annotations associated with function arguments and return types, throughout the...

Potentially Unused Function plot_ori_map

1

The [variable `roi`](https://github.com/HEXRD/hexrd/blob/d57ef03b96abb1c9536d41add40f84606c63dd49/hexrd/grainmap/nfutil.py#L1480) is undefined in this function. This leads me to believe that the function has never been used, and probably won't be. Consider deletion.

Potentially Unused Function _gvec_to_detector_array

3

The function's [documentation](https://github.com/HEXRD/hexrd/blob/d57ef03b96abb1c9536d41add40f84606c63dd49/hexrd/grainmap/nfutil.py#L363-L372) claims to be identical to `gvec_to_xy` from the transforms module, but worse. It also doesn't seem to be used in all of hexrd. Consider deletion.