Skip to content

Commit

Permalink
apply suggestions from code review - comments
Browse files Browse the repository at this point in the history
Co-authored-by: Hendrik Ranocha <ranocha@users.noreply.github.com>
  • Loading branch information
huiyuxie and ranocha authored May 3, 2024
1 parent 12128a8 commit 388917b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/equations/acoustic_perturbation_2d.jl
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ function initial_condition_convergence_test(x, t,
RealT = eltype(x)
a = 1
c = 2
L = convert(RealT, 2)
L = convert(RealT, 2) # since we divide by L below
f = 2 / L
A = convert(RealT, 0.2)
omega = 2 * convert(RealT, pi) * f
Expand Down Expand Up @@ -158,7 +158,7 @@ function source_terms_convergence_test(u, x, t,
RealT = eltype(u)
a = 1
c = 2
L = convert(RealT, 2)
L = convert(RealT, 2) # since we divide by L below
f = 2 / L
A = convert(RealT, 0.2)
omega = 2 * convert(RealT, pi) * f
Expand Down
4 changes: 2 additions & 2 deletions src/equations/compressible_euler_1d.jl
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ function initial_condition_convergence_test(x, t,
RealT = eltype(x)
c = 2
A = convert(RealT, 0.1)
L = convert(RealT, 2)
L = convert(RealT, 2) # since we divide by L below
f = 1 / L
ω = 2 * convert(RealT, pi) * f
ini = c + A * sin* (x[1] - t))
Expand All @@ -97,7 +97,7 @@ Source terms used for convergence tests in combination with
RealT = eltype(u)
c = 2
A = convert(RealT, 0.1)
L = convert(RealT, 2)
L = convert(RealT, 2) # since we divide by L below
f = 1 / L
ω = 2 * convert(RealT, pi) * f
γ = equations.gamma
Expand Down

0 comments on commit 388917b

Please sign in to comment.