fmihpc / vlasiator

Vlasiator - ten letters you can count on
https://www.helsinki.fi/en/researchgroups/vlasiator
Other
47 stars 39 forks source link

Include some form of Dispersion tests in the testpackage #278

Open ykempf opened 8 years ago

ykempf commented 8 years ago

Dispersion is very sensitive to anything done to the solvers, as the Vlasiator Summer of Debugging 2016 has shown.

To avoid having to put in the extra code, a version of the Dispersion tools suite using analysator would be very useful.

General comment from @ykempf: beware of the file flood when doing the FFT, ensure they get removed after the test as they will likely be big.

galfthan commented 8 years ago

We could perhaps add a new function/hook into projects, that enable them to do some custom processing/IO during the simulation. In base class it would do nothing, but for dispersion it could write out the more efficient binary ones.

ykempf commented 8 years ago

Well that means an extra piece of code in the main propagation loop...

ykempf commented 8 years ago

I take Urs' comment on Flowdock as a volunteering to be assigned this issue.

ursg commented 8 years ago

I was just about to comment here. Indeed. :)

galfthan commented 8 years ago

It would be one extra line, so not too bad. My phiprof merge cleans out more lines than that ;) A project in itself would be to clean up main() to make it self explanatory and failry easy for somebody to understand