Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add README to KHI test #4761

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

PrometheusPi
Copy link
Member

The KHI test was missing a README file. This pull request adds a short description?
@psychocoderHPC should I also add a quick example on how to run it?

@PrometheusPi PrometheusPi added documentation regarding documentation or wiki discussions component: tests unit tests labels Dec 5, 2023
@PrometheusPi PrometheusPi added this to the 0.8.0 / Next stable milestone Dec 5, 2023
@PrometheusPi PrometheusPi added the CI:no-compile CI is skipping compile/runtime tests but runs PICMI tests label Dec 5, 2023
Copy link
Member

@steindev steindev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see comments.

share/picongpu/tests/KHI_growthRate/README.rst Outdated Show resolved Hide resolved
@steindev
Copy link
Member

steindev commented Dec 6, 2023

And is this text supposed to appear in the readthedocs? By now it is not included.

@psychocoderHPC
Copy link
Member

@PrometheusPi Could you please finish this PR.

Copy link
Member

@steindev steindev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some typos remain. Please squash all commits before next push.

.. sectionauthor:: Richard Pausch <r.pausch (at) hzdr.de>
.. moduleauthor:: Richard Pausch <r.pausch (at) hzdr.de>, Mika Soren Voß <m.voss (at) hzdr.de>

This KHI growth rate test simulates a shear-flow instability known as the Kelvin-Helmholtz Instability in a sub-relativistic setup as studied in [Alves12]_, [Grismayer13]_, [Bussmann13]_. The setup uses a pre-ionized quasi-neutral hydrogen plasma. From the simulation, the growth of the dominant magnetic field is extracted and compared to theoretic predictions. The tests only passes if the simulation reaches the ideal theoretical growth rate with a 10% accuracy.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

theroretic -> theoretical
The tests only passes ... -> The tests only pass ...


.. [Grismayer13]
T. Grismayer, E.P. Alves, R.A. Fonseca, L.O. Silva.
*dc-magnetic-field generation in unmagnetized shear flows*,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Phys Rev writes the title as 'dc-Magnetic-Field Generation in Unmagnetized Shear Flows'.

@PrometheusPi
Copy link
Member Author

@steindev I added all your suggestions. And I added a comment on the need of $PICSRC which we found out today with @psychocoderHPC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI:no-compile CI is skipping compile/runtime tests but runs PICMI tests component: tests unit tests documentation regarding documentation or wiki discussions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants