search

pseudospectators / FLUSI

Fluid-Structure-Interaction / MHD Research Code. Relies on Fourier transforms for derivatives and the volume penalization method to include obstacles.
http://aifit.cfd.tu-berlin.de/
GNU General Public License v3.0
46 stars 29 forks source link

Order of loops #7

Closed dkolom closed 9 years ago

dkolom commented 9 years ago

Loops like do ix=ra(1),rb(1) do iy=ra(2),rb(2) do iz=ra(3),rb(3) ... should be changed to do iz=ra(3),rb(3) do iy=ra(2),rb(2) do ix=ra(1),rb(1) ... for better efficiency

tommy-engels commented 9 years ago

I solved it in flusi_tuning, but it still has to be merged

dkolom commented 9 years ago

Well... I did the same in master and ran some tests. Everything looks fine so it is solved. But we should be careful when merging.

On Sun, Dec 21, 2014 at 11:54 AM, Thomas Engels notifications@github.com wrote:

I solved it in flusi_tuning, but it still has to be merged

— Reply to this email directly or view it on GitHub https://github.com/pseudospectators/FLUSI/issues/7#issuecomment-67766565 .

dkolom commented 9 years ago

Sorry for modifying the masters branch directly! Normally I shouldn't have done that, but I was very upset about this issue.