Mark OLESEN authored 4 years ago and Andrew Heather committed 4 years ago

- support selective enable/disable of the file banner.

ENH: improve code isolation for decomposedBlockData

- use readBlockEntry/writeBlockEntry to encapsulate the IO handling,
  which ensures more consistency

- new decomposedBlockData::readHeader for chaining into the
  block header information.

- remove unused constructors for decomposedBlockData

ENH: minor cleanup of collated fileOperations

Mark OLESEN authored 4 years ago and Andrew Heather committed 4 years ago

- improves interface and data consistency.
  Older signatures are still active (via the Foam_IOstream_extras
  define).

- refine internals for IOstreamOption streamFormat, versionNumber

ENH: improve data alignment for IOstream and IOobject

- fit sizeof label/scalar into unsigned char

STYLE: remove dead code

Mark OLESEN authored 4 years ago and Andrew Heather committed 4 years ago

- read/write lists of character data in binary only.
  This is the only means of preserving data.
  If character data are written as an ASCII list, there is no means of
  determining if spaces or newlines are content or separators.

Mark OLESEN authored 4 years ago and Andrew Heather committed 4 years ago

- handle binary/contiguous first as being the most obvious, followed
  by increasing complexity of ascii.
  Structure reading and writing routines similarly by introducing a
  readList method to compliment the writeList method.

Mark OLESEN authored 4 years ago and Andrew Heather committed 4 years ago

- would ideally like the same for tmp, but there is too much existing
  code that uses a multiply operation that would be interpreted as a
  dereference.

- generally OK to miss for easily evaluated functions, but highly
  misleading with expressions (where the evaluate may be invalid
  during construction).

- based on fix supplied by Henning.Scheufler@dlr.de

STYLE: adjust wording of age-warning

- override casename, procesorCase flags to guarantee reconstructed
case to be written to the undecomposed directory
- alternative is to construct a Zero mesh on the undecomposed
runTime and add all other bits to that but that has not been
pursued

- patch point ordering was affected by the EnSight face sorting
  (tri, quad, nsided). This did not affect other point fields,
  since they use the internal field values.

TUT: lagrangian: clean up tutorials

See merge request !429

- fvMeshDistribute and redistributePar changes

- for use when the is_contiguous check has already been done outside
  the loop. Naming as per std::span.

STYLE: use data/cdata instead of begin

ENH: replace random_shuffle with shuffle, fix OSX int64 ambiguity

- direct check of punctuation.
  For example,

      while (!tok.isPunctuation(token::BEGIN_LIST)) ..

  instead of

  while (!(tok.isPunctuation() && tok.pToken() == token::BEGIN_LIST)) ..

  Using direct comparison (tok != token::BEGIN_LIST) can be fragile
  when comparing int values:

      int c = readChar(is);
      while (tok != c) ..  // Danger, uses LABEL comparison!

- direct check of word.
  For example,

      if (tok.isWord("uniform")) ..

  instead of

      if (tok.isWord() && tok.wordToken() == "uniform") ..

- make token lineNumber() a setter method

ENH: adjust internal compound method empty() -> moved()

- support named compound tokens

STYLE: setter method for stream indentation

- orient blocks in global x/y/z.
  Provisioning for coarser/smaller geometries.

- single-step for reconstructParMesh
- no point merging for redistributePar -reconstruct

COMP:  objective manager masks regIOobject::write function (#2005)

See merge request !427

- int64 ambiguity
- std::array include

- bsd-sed syntax (replaces gnu-sed syntax):
  * wmake-build-info
  * wmake-with-bear

Kutalmış Berçin authored 4 years ago and Mark OLESEN committed 4 years ago

- return type of `eigenVectors`  field functions should be
  `tensor` instead of `symmTensor`

First receives all mesh parts and then stitches them
all together in one step. This avoids lots of intermediate
addressing calculation

Thermo ext

See merge request Development/openfoam!424

sergio authored 4 years ago and Kutalmış Berçin committed 4 years ago

Adding thermo options for mass exchange Lee model for multiPhaseInter using
tabulated type of thermos.

- pointPatches may or may not have a "value" type.
  Use the patch value field where possible and the internal field
  otherwise. Previously always used the internal field.

- the raw surface writer simply outputs x/y/z and field values.
  This additional flag allows recovery of some geometric information.

- optional user-specified output precision

Example,
```
formatOptions
{
    raw
    {
        normal      yes;
        precision   10;
    }
}
```

Kutalmış Berçin authored 4 years ago and Mark OLESEN committed 4 years ago

- centralises existing functions (erfInv, incGamma*, invIncGamma*).
  Provides a location for additional functions in the future.

- adjusted existing models to use these functions
  (e.g. distributionModels::normal)

- location and naming more consistent with other controls

    old: DebugSwitch  'level'
    new: InfoSwitch   'outputLevel'

- effectively 'steals' the pointer from the table but leaves its
  name as a placeholder