Sergio Ferraris authored 4 years ago and Andrew Heather committed 4 years ago

walls

A new user input parameter UrMax is added to the PatchInteractionModel.
In some occasions the partile remains on a patch face due to extremely
low relative U. If this Ur is lower than UrMax the particle is removed

- previously introduced `getOrDefault` as a dictionary _get_ method,
  now complete the transition and use it everywhere instead of
  `lookupOrDefault`. This avoids mixed usage of the two methods that
  are identical in behaviour, makes for shorter names, and promotes
  the distinction between "lookup" access (ie, return a token stream,
  locate and return an entry) and "get" access (ie, the above with
  conversion to concrete types such as scalar, label etc).

    Adding check for p.active at the end of KinematicParcel::move.
    p.hitFace() is called only for active parcels.

    Setting to zero the initialization for stored lists of stick and
escape parcels
    in LocalInteraction and StandardWallInteraction models

    NOTE: KinematicParcel::hitPatch counts overall system escaped
    parcels and mass based on polyPatch type and not on type of
    patchInteractionModel. Thus, if a patch is a Wall for fluid
    but escape for parcel the overall report will be wrong but
    the local report for each patch is correct

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

File writing is off by default; to activate, add to the patch interaction model
coeff dictionary

    writeToFile yes;

- should have been limited to non-processor patches only

Will Bainbridge authored 7 years ago and Andrew Heather committed 7 years ago

A lot of methods were taking argument data which could be referenced or
generated from the parcel class at little or no additional cost. This
was confusing and generated the possibility of inconsistent data states.

Will Bainbridge authored 7 years ago and Andrew Heather committed 7 years ago

The KinematicCloud::patchData method has been made consistent on moving
meshes and/or when the time-step is being sub-cycled.

It has also been altered to calculate the normal component of a moving
patch's velocity directly from the point motions. This prevents an
infinite loop occuring due to inconsistency between the velocity used to
calculate a rebound and that used when tracking.

Some minor style improvements to the particle class have also been made.

coupled patches, to prevent rebound/stick/etc... on these patches. Also
added "none" interaction type to LocalInteraction, which reverts the
patch interaction to the fundamental behaviour. This is primarily useful
for non-coupled constraint types.

Resolves https://bugs.openfoam.org/view.php?id=2458

to have the prefix 'write' rather than 'output'

So outputTime() -> writeTime()

but 'outputTime()' is still supported for backward-compatibility.

Also removed the redundant secondary-writing functionality from Time
which has been superseded by the 'writeRegisteredObject' functionObject.

- assists in decoding what the binary IO content means

ENH: use label instead of bool for the KinematicParcel active state (fixes #111)

- avoids internal padding of the data structure and simplifies
  downstream use.

ENH: make particle sizeofFields public (fixes #110)

Also fixes #108 (missing faceI and stepFraction entries) which had
also been fixed in the upstream as well.

Do not set accumulated properties to 0 on output

Updates provided by Karl Meredith

Avoids the clutter and maintenance effort associated with providing the
function signature string.

This reverts commit c5bea524.

This reverts commit b18f6cc1.

sub-models:
- now derived from a common base class
- copy/clone functionality added

cloud:
- makes use of cachedRandom class (instead of Random)