You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I notice that in line 148 of telescope_functions.py
all_rotated_Nbase = np.array([earth_rotation_effect(Nbase, i, int_time, declination) for i in time_indices])
you create an array which contain all baselines for all time points which could be rather large. It seems like that making it a generator, i.e.
all_rotated_Nbase =(earth_rotation_effect(Nbase, i, int_time, declination) for i in time_indices)
would be more efficient for memory usage because the baseline vectors would only be created when they are going to be used.
In following calculation you just loop over time indices and do not use the time dimension in a vectorized way. I cannot see the improvement on speed by making it an array.
The text was updated successfully, but these errors were encountered:
I notice that in line 148 of telescope_functions.py
you create an array which contain all baselines for all time points which could be rather large. It seems like that making it a generator, i.e.
would be more efficient for memory usage because the baseline vectors would only be created when they are going to be used.
In following calculation you just loop over time indices and do not use the time dimension in a vectorized way. I cannot see the improvement on speed by making it an array.
The text was updated successfully, but these errors were encountered: