Skip to content

Commit

Permalink
modify/remove the assertions about no hidden dimension since this cas… (
Browse files Browse the repository at this point in the history
#4180)

…e is working in ERF

## Summary

## Additional background

## Checklist

The proposed changes:
- [ ] fix a bug or incorrect behavior in AMReX
- [ ] add new capabilities to AMReX
- [ ] changes answers in the test suite to more than roundoff level
- [ ] are likely to significantly affect the results of downstream AMReX
users
- [ ] include documentation in the code and/or rst files, if appropriate
  • Loading branch information
asalmgren authored Oct 3, 2024
1 parent 8349789 commit f3514e4
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions Src/LinearSolvers/MLMG/AMReX_MLCellLinOp.H
Original file line number Diff line number Diff line change
Expand Up @@ -536,7 +536,11 @@ MLCellLinOpT<MF>::setLevelBC (int amrlev, const MF* a_levelbcdata, const MF* rob
{
if (this->needsCoarseDataForBC())
{
AMREX_ALWAYS_ASSERT(!this->hasHiddenDimension());
// AMREX_ALWAYS_ASSERT(!this->hasHiddenDimension());
if (this->hasHiddenDimension()) {
int hidden_dir = this->hiddenDirection();
AMREX_ALWAYS_ASSERT(this->m_coarse_data_crse_ratio[hidden_dir] == 1);
}
br_ref_ratio = this->m_coarse_data_crse_ratio.allGT(0) ? this->m_coarse_data_crse_ratio : IntVect(2);
if (m_crse_sol_br[amrlev] == nullptr && br_ref_ratio.allGT(0))
{
Expand Down Expand Up @@ -1946,8 +1950,6 @@ MLCellLinOpT<MF>::computeVolInv () const
m_volinv[amrlev].resize(this->NMGLevels(amrlev));
}

AMREX_ASSERT(this->m_coarse_fine_bc_type == LinOpBCType::Dirichlet || ! this->hasHiddenDimension());

// We don't need to compute for every level

auto f = [&] (int amrlev, int mglev) {
Expand Down

0 comments on commit f3514e4

Please sign in to comment.