ENH: downgrade error on type-mismatch to a warning

- Ensight data files generated by OpenFOAM will start with the name of
  the data type (Eg, "scalar", "vector" etc), but this convention may
  fail for data files that have been generated elsewhere.

mattijs authored 5 years ago and Mark OLESEN committed 5 years ago

edge it sets the globalEdgeOrientation

See #1367

mattijs authored 5 years ago and Mark OLESEN committed 5 years ago

- synchronization, reduction only makes sense on processor-coupled
  patches. Since cyclic baffles are within a single processor domain,
  they are not reduced. So need to skip the sanity test for these.

- the read offset missed the tag_ member entirely and thus the entire
  particle information would be corrupt (incorrectly interpreted) as
  well as potential violation of adjacent (trailing) memory locations.

- now suppress any '#line' if the input number number is invalid
  (ie, an empty set of tokens)

- need to be more generous when prefiltering the cell selection

- now catch these and emit a warning.
  Still need to investigate the root cause in the caller(s) or regionSplit.

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

BUG: DimensionedField - register new field if it does not have the same name as the copy - see #1348

- was missing range setting for the colour lookup table

- the emissivity was ignored for 'hpTa' when there is no solid resistance

STYLE: adjust debug statement in externalWallHeatFlux

- used fallback of 0 instead of the results time.
  This discrepancy caused the case file to have two timesets that
  only differed by the first (incorrect) entry.

pointNormals calculation gets triggered through processorfvPatches
so on processors that don't have these bypass this. This leads
to the global reduction hanging.

- retrieved value for the modelType was masked by a local variable

- checkIn() was incorrectly conditional on the register state of the
  source object.

Partial patch from .org commit 6dc48b62d948

- could be triggered if running in parallel, but requesting procIDs
  to be written with a non-parallel version of the writer.

- uses the value of foamVersion::api, which should be reliable.

- they had an isType<> instead of isA<>() check