Manual fixes: alpha for thermal expansivity and more #1095
Conversation
… Rayleigh number in Section 5.3.1, Simple convection in a quarter of a 2d annulus
you don't need to open a new PR but you can update the old one by commiting and pushing. |
| = | ||
| \frac{10\, \text{m}\,\text{s}^{-2} \times 4\cdot 10^{-5}\text{K}^{-1} \times 3300\, | ||
| \text{K} \times 3300\, \text{kg}\,\text{m}^{-3} \times (2.86 \cdot 10^6 | ||
| \text{m})^3}{10^{-6}\, \text{m}^2\,\text{s}^{-1}\times 10^{22}\, |
There was a problem hiding this comment.
can you explain this change? It is somewhat hard to read without seeing the pdf. It looks like the exponent of m changed?
There was a problem hiding this comment.
Looking at this again, it turns out that the only non-obvious part is the addition of the factor 10^6 m^2/s. This is because the original formula was simply missing it -- it corresponds to the thermal diffusivity kappa.
tjhei
changed the title
|
I am refining the wording in section 5.3.1. I will submit a revision when that is complete. |
|
I hope you had a conversation with @bangerth about the units in the RA computation. It would be great to continue that equation and provide an estimate for the RA number. |
|
Yes, we talked about it. I gave @lkellogg permission to do with the section as she pleases because she was clearly right :-) |
|
@lkellogg -- any progress here? We'd love to have you as an author ;-) |
|
See also #1304. |
|
Also see #989. |
|
I think that upon taking a closer look, this is actually correct. |
ghost
deleted the
This is entirely justified following the merge of geodynamics#1095.
Added space in \times10. This supersedes #1094