Skip to content

Commit

Permalink
Update solve.inl
Browse files Browse the repository at this point in the history
Added some explanation on the ApproximationFunction argument
  • Loading branch information
AdityaDendukuri authored Aug 4, 2024
1 parent d91f79c commit 0f82e21
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions include/tuvx/radiative_transfer/solvers/solve.inl
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,9 @@ namespace tuvx
tuvx::InitializeVariables<T, GridPolicy, ProfilePolicy, RadiatorStatePolicy, RadiationFieldPolicy>(
solar_zenith_angles, grids, profiles, accumulated_radiator_state);

// Currently only the Delta Approximation is implemented in delta_eddington.inl
// There are other approximations defined in Table 1 of the paper
// TODO : Move this comment to the docstring in the header.
ApproximationFunction(accumulated_radiator_state, solar_zenith_angles, solver_variables);

tuvx::AssembleTridiagonalMatrix<T>(
Expand Down

0 comments on commit 0f82e21

Please sign in to comment.