Draft: Added loop advection test, should run out of the box

added MHD label

requested review from @fastasyszyn

assigned to @matthieu

added 1 commit

• 787b713e - Python script formatting

Compare with previous version

Works out of the box. Nice!

Just to check, do we need that high a resolution by default?

I have no major comment than the resolution, and maybe more explanatory information in the Readme? what do we expect from the test?

approved this merge request

Any updates on the questions above?

The resolution chosen is in the realm of what other people do. Why do you think 2*32**3 cubed particles is a lot ? Also, regarding the README, what would you like me to add ? In all other tests' READMEs we link papers where each problem is described, but never explicitly elaborate on what each individual test is / what we expect from it.

README is fine as-is, thanks.

As for the resolution, I was only asking to see if we can simplify things. Fewer particles means faster to run, so good if it is a quick check we like to run often to test models. If the current resolution is necessary to see the features we want to see then it's all good like this. Just let me know.

I'll triple check whether lower resolution gives us enough to differentiate between models. Indeed, better to keep that as the default then

Any news?

Any news?

I'll set this as draft until updates are brought.

marked this merge request as draft

Any updates on the questions above? Or shall we close?

@okarapiperis any updates on the points above?

Closing as no activity.

closed