Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 isotropic viscosity and resistivity #89
Add isotropic viscosity and resistivity #89
Changes from 65 commits
b5baf67
e9ac8f6
2b30fca
db63255
a968fd8
0e0986b
6da6766
d25abb3
b4dfdd9
5f20625
b75b496
3473da8
3d26c40
61a9c93
647490d
4e03d38
a235952
991d353
7fd557d
755e06b
c2acc9e
1f9cf68
1b9a797
16ad2ca
c9f1dfb
8599682
4ebb5d6
8800b51
b416ce0
3986cee
3f071d4
9582764
115285d
51dca4f
8194e97
ca3c397
6494ab2
6df018f
1d0cb19
300c6a0
9205e56
1749bb8
2bc56f8
3b6ac2e
b0bd7ca
4ab8834
3199c6a
42f9f3c
1c51435
5fea712
2cd1e60
93fa27f
bd3834f
6e0151e
98f2f1f
17f5a69
3f788ad
3e18c25
406ec30
07fc229
5319ff3
45165ff
3ab1c95
8e1910e
a9e5203
3be0a24
043a028
4e41e6d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
again, for rkl2/operator split integrator (the main point here is that there are a lot of knobs to turn and it'd help to avoid confusion in this regard)
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.
Actually this applies to both. I adjusted the comment accordingly.
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.
A general comment about coefficients - based on conversations with my current grad students, they would really appreciate if the units of the viscosity and resistivity were mentioned in the documentation, and perhaps a reference (even to a Shu or Spitzer textbook section) where they can figure out some details about the physics so they can set numbers appropriately.
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 just added the info on the units itself.
Regarding text book references, I'm happy if any student who does a deep dive intro opens a PR to extend the documentation accordingly.
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.
this would be a helpful place to mention the units of viscosity, even if it's only available in code units (this would help students who are trying to figure out what values are reasonable based on other simulation settings). Same thing applies for the resistivity subsection below.
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.
Added info both here and in the overview block above.