Skip to content

Commit

Permalink
Merge pull request #5264 from anne-glerum/check_for_requested_propert…
Browse files Browse the repository at this point in the history
…y_viscosity

Remove todo to assert that viscosity has been requested and computed
  • Loading branch information
gassmoeller committed Jul 11, 2023
2 parents 3d823e0 + f00fa30 commit e499804
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions source/material_model/visco_plastic.cc
Original file line number Diff line number Diff line change
Expand Up @@ -278,8 +278,7 @@ namespace aspect

// Fill plastic outputs if they exist.
// The values in isostrain_viscosities only make sense when the calculate_isostrain_viscosities function
// has been called. TODO check here for in.requests_property(MaterialProperties::viscosity) or
// in fill_plastic_outputs as is done now?
// has been called.
// TODO do we even need a separate function? We could compute the PlasticAdditionalOutputs here like
// the ElasticAdditionalOutputs.
rheology->fill_plastic_outputs(i, volume_fractions, plastic_yielding, in, out, isostrain_viscosities);
Expand Down

0 comments on commit e499804

Please sign in to comment.