Conversation
gknapp1
left a comment
There was a problem hiding this comment.
Quick review while still in draft state. Looks good from reading through the code, but request a re-review once this exist draft state and I'll test it.
If you have any references for the origin of the mathematical form, would be good to include them in the source code somewhere for documentation.
...solvers/additiveFoam/movingHeatSource/heatSourceModels/projectedGaussian/projectedGaussian.C
Show resolved
Hide resolved
We don't have a citation for the mathematical form yet. It is the focus of a manuscript I am currently preparing. |
|
Approved. For testing, I used the implemented heat source with the |
Projects gaussian heat source which uses a supergaussian axial decay function, where the coefficient on the decay function can be specified as a function of the heat source aspect ratio (a), such that: n = A*log2(a) + B