search

shbhuk / barycorrpy

Python version of Barycorr
GNU General Public License v3.0
37 stars 6 forks source link

True vectorizing? #11

Closed mzechmeister closed 6 years ago

mzechmeister commented 6 years ago

Currently there is a for-loop over BCPy. Maybe it is possible to vectorize it as the IDL version? Might be also useful for #6.

shbhuk commented 6 years ago

Hi, Not changing this for now in case there is something else that needs to be added in the for loop. We decided to implement #6 as a standalone function, so as to not break the get_BC_vel function for existing users.