Skip to content

Commit

Permalink
fix dumb spelling mistakes
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewwinters5000 committed May 22, 2024
1 parent 21ae0cb commit c39dc48
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/solvers/scratch_p4est.jl
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

# TODO: once working the mortar methods could likely be extended to the other
# equations types available in the package. Although for the multilayer equations
# care must be taken beacuse the pressure term is separated from the physical flux
# care must be taken because the pressure term is separated from the physical flux
# and directly placed in the nonconservative flux

# The methods below are specialized on the mortar type
Expand Down Expand Up @@ -118,7 +118,7 @@ function Trixi.prolong2mortars!(cache, u,
# After the projection of the constant solution we can modify the values
# in the first solution variable to no longer be the sigma variable of
# Benov et al. and instead be the conservative water height variable `h`.
# Bascially, unpacking the sigma variable to create the projected local water
# Basically, unpacking the sigma variable to create the projected local water
# height from Eq. 41 in Benov et al.
# TODO: My main hope was that this avoids allocations
# TODO: My other hope is that such a strategy will make this code extensible
Expand Down Expand Up @@ -308,7 +308,7 @@ end

# Project small numerical fluxes and physical flux penalty computed on the projected
# large element solution back onto large element.
# This is bascially Eq. (46) from Benov et al. where the factor of 1/2 is already
# This is basically Eq. (46) from Benov et al. where the factor of 1/2 is already
# already included in `reverse_upper` and `reverse_lower` operators.
Trixi.multiply_dimensionwise!(u_buffer,

Check warning on line 313 in src/solvers/scratch_p4est.jl

View check run for this annotation

Codecov / codecov/patch

src/solvers/scratch_p4est.jl#L313

Added line #L313 was not covered by tests
mortar_l2.reverse_upper, (fstar_secondary[2] .- f_large[2]),
Expand All @@ -332,7 +332,7 @@ end
large_direction = Trixi.indices2direction(large_indices)

Check warning on line 332 in src/solvers/scratch_p4est.jl

View check run for this annotation

Codecov / codecov/patch

src/solvers/scratch_p4est.jl#L330-L332

Added lines #L330 - L332 were not covered by tests

# TODO: We need to store the unprojected solution in the mortars such that we have access to them
# here when we go to compute the flux on the parent elements and remove the physcial flux evaluated
# here when we go to compute the flux on the parent elements and remove the physical flux evaluated
# at the upprojected solution state that is present from the volume integral computation
index_range = eachnode(dg)
i_large_start, i_large_step = Trixi.index_to_start_step_2d(large_indices[1],

Check warning on line 338 in src/solvers/scratch_p4est.jl

View check run for this annotation

Codecov / codecov/patch

src/solvers/scratch_p4est.jl#L337-L338

Added lines #L337 - L338 were not covered by tests
Expand Down

0 comments on commit c39dc48

Please sign in to comment.