search

manodeep / Corrfunc

⚡️⚡️⚡️Blazing fast correlation functions on the CPU.
https://corrfunc.readthedocs.io
MIT License
163 stars 50 forks source link

Drop certain code options #267

Open manodeep opened 2 years ago

manodeep commented 2 years ago

While reviewing #258, I realised that perhaps we can retire options like fast_acos, and fast_divide from the code-base. Not sure if we would leave any users out though (or how to go about identifying users that do need those features)

@lgarrison Do you have any thoughts on this?

lgarrison commented 2 years ago

I've never looked at that part of the code very carefully! On the surface, they sound like useful code options; are they deprecated/not useful in some way?