Remove calc_gaussian_stirring_potential from BoseEinsteinCondensate.py

Question

Opened this issue 3 months ago · 3 comments

There is already a calc_Gaussian function in the base_system_calc file. COmputing it twice is redundant.

Answer 1 · 2024-07-23T02:23:44.000Z

Do you suggest removing it completely or just make it cal the function in the base_system_calc file?

Answer 2 · 2024-07-23T05:38:04.000Z

I think we should remove it completely and replace all instances where it is called by calling the corresponding function in the base_system class.

Answer 3 · 2024-07-23T05:49:50.000Z

Careful when changing though. The width kwarg in bs.calc_Gaussian is not exactly the same as the size kwarg in calc_gaussian_stirring_potential