BOUT-dev icon indicating copy to clipboard operation
BOUT-dev copied to clipboard

Replace Mesh::iterateBndry* functions with Regions

Open ZedThree opened this issue 6 years ago • 3 comments

The following Mesh methods should all be deprecated and replaced with Regions:

  virtual const RangeIterator iterateBndryLowerY() const = 0;

  /// Iterate over the upper Y boundary
  virtual const RangeIterator iterateBndryUpperY() const = 0;
  virtual const RangeIterator iterateBndryLowerOuterY() const = 0;
  virtual const RangeIterator iterateBndryLowerInnerY() const = 0;
  virtual const RangeIterator iterateBndryUpperOuterY() const = 0;
  virtual const RangeIterator iterateBndryUpperInnerY() const = 0;

ZedThree avatar Mar 20 '18 11:03 ZedThree

Some of this can be done with PR #972 but the rest need offsets. Also the handling of Y boundary regions needs thinking about.

bendudson avatar Apr 02 '18 08:04 bendudson

I think this should be achievable with the current implementation of Region in next.

d7919 avatar Sep 03 '18 11:09 d7919

See #951 why this is tricky

ZedThree avatar Oct 11 '18 14:10 ZedThree