-
Notifications
You must be signed in to change notification settings - Fork 218
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
allow constructor for density free formular functor #3024
allow constructor for density free formular functor #3024
Conversation
`FreeFormulaImpl` allows the user functor to define a constructor to move additional information into the profile. This allow e.g. filtering by cell position relative to the global volume
Great work - I will test it right away. |
Looks good to me, not sure if @PrometheusPi has tried this out. |
@psychocoderHPC and @sbastrakov sorry I was busy with creating videos and did not test it yet - will do this tomorrow |
I am testing it right now. |
@PrometheusPi how did the test go? :) |
The issue seen in the isaac visualization was caused by an error on my input set and the strange behavior of the automatic range setting (see ComputationalRadiationPhysics/isaac#99). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Is the only algorithmic difference, that the constructor is now called by the template constructor?
@PrometheusPi on your constructor question - yes, the idea is that with this PR one can pass arbitrary parameters to a constructor of the user-provided functor through the constructor of Edit: upon a further look, it is not actually arbitrary but the choice is only between the default constructor and the one taking a |
@sbastrakov Thank you for the explanation. |
@ComputationalRadiationPhysics/picongpu-maintainers after this pull request has been added - should the KHI example be refactored to use 4 instead of 2 species? |
@sbastrakov Since @psychocoderHPC is on vacation, I quickly fixed the typo. |
discussed offline with @sbastrakov: |
FreeFormulaImpl
allows the user functor to define a constructor to move additional information into the profile.This allow e.g. filtering by cell position relative to the global volume
example