-
Notifications
You must be signed in to change notification settings - Fork 24
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
Integrate particle source #84
Conversation
Hopefully, the regression test is fixed. |
@fsciortino I need the approval of this pull request. I'm not allowed to merge it before approval. All regression tests have passed, so hopefully the installation issues are fixed. |
aurora/adas_files.py
Outdated
@@ -32,7 +32,7 @@ | |||
) | |||
|
|||
|
|||
def get_adas_file_loc(filename, filetype="adf11"): | |||
def get_adas_file_loc(filename, filetype="adf11", tmp_folder=None): |
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.
@odstrcilt why do you need this tmp_folder
here?
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.
I have no idea. I will fix it.
@fsciortino ready to be merged |
Integrate particle source over the timestep of the simulation. It will reduce particle conservation error for noisy or fast varying impurity sources.