CONFIG: change default distanceSurface algorithm from 'cell' to 'topo'

- yields cleaner surfaces with few cuts.

  Can use isoMethod keyword to select cell/point/topo if they prove
  better for any particular case.

CONFIG: change default cuttingPlane algorithm from 'cell' to 'topo'
This commit is contained in:
Mark Olesen
2020-11-30 23:03:30 +01:00
parent b1a27d3d00
commit 811a83599e
6 changed files with 11 additions and 10 deletions

View File

@ -50,7 +50,7 @@ Usage
type | distanceSurface | yes | type | distanceSurface | yes |
distance | Distance from surface | yes | distance | Distance from surface | yes |
signed | Use sign when distance is positive | partly | signed | Use sign when distance is positive | partly |
isoMethod | Iso-algorithm (cell/topo/point) | no | cell isoMethod | Iso-algorithm (cell/topo/point) | no | topo
regularise | Point snapping for iso-surface | no | true regularise | Point snapping for iso-surface | no | true
average | Cell values from averaged point values | no | false average | Cell values from averaged point values | no | false
bounds | Limit with bounding box | no | bounds | Limit with bounding box | no |

View File

@ -366,7 +366,7 @@ Foam::sampledCuttingPlane::sampledCuttingPlane
isoParams_ isoParams_
( (
dict, dict,
isoSurfaceParams::ALGO_POINT, isoSurfaceParams::ALGO_TOPO,
isoSurfaceParams::filterType::DIAGCELL isoSurfaceParams::filterType::DIAGCELL
), ),
average_(dict.getOrDefault("average", false)), average_(dict.getOrDefault("average", false)),

View File

@ -56,8 +56,8 @@ Usage
type | cuttingPlane | yes | type | cuttingPlane | yes |
planeType | plane description (pointAndNormal etc) | yes | planeType | plane description (pointAndNormal etc) | yes |
mergeTol | tolerance for merging points | no | 1e-6 mergeTol | tolerance for merging points | no | 1e-6
isoMethod | Iso-algorithm (cell/topo/point) | no | point isoMethod | Iso-algorithm (cell/topo/point) | no | topo
regularise | point snapping (bool or enum) | no | true regularise | Face simplification (enum or bool) | no | true
bounds | limit with bounding box | no | bounds | limit with bounding box | no |
zone | limit to cell zone (name or regex) | no | zone | limit to cell zone (name or regex) | no |
zones | limit to cell zones (names, regexs) | no | zones | limit to cell zones (names, regexs) | no |

View File

@ -76,8 +76,7 @@ Foam::distanceSurface::distanceSurface
isoParams_ isoParams_
( (
dict, dict,
isoSurfaceParams::ALGO_CELL, isoSurfaceBase::ALGO_TOPO
isoSurfaceParams::filterType::DIAGCELL
), ),
isoSurfCellPtr_(nullptr), isoSurfCellPtr_(nullptr),
isoSurfPointPtr_(nullptr), isoSurfPointPtr_(nullptr),

View File

@ -38,8 +38,8 @@ Usage
Property | Description | Required | Default Property | Description | Required | Default
distance | distance from surface | yes | distance | distance from surface | yes |
signed | Use sign when distance is positive | partly | signed | Use sign when distance is positive | partly |
isoMethod | Iso-algorithm (cell/topo/point) | no | cell isoMethod | Iso-algorithm (cell/topo/point) | no | topo
regularise | Point snapping (enum or bool) | no | true regularise | Face simplification (enum or bool) | no | true
bounds | Limit with bounding box | no | bounds | Limit with bounding box | no |
surfaceType | Type of surface | yes | surfaceType | Type of surface | yes |
surfaceName | Name of surface in \c triSurface/ | no | dict name surfaceName | Name of surface in \c triSurface/ | no | dict name
@ -57,6 +57,8 @@ Note
The keyword \c cell (bool value) which was use in 1906 and earlier to switch The keyword \c cell (bool value) which was use in 1906 and earlier to switch
between point/cell algorithms is now ignored (2020-12). between point/cell algorithms is now ignored (2020-12).
Changed default algorithm from cell to topo (2020-12).
SourceFiles SourceFiles
distanceSurface.C distanceSurface.C
@ -82,7 +84,7 @@ namespace Foam
class distanceSurface class distanceSurface
{ {
// Private data // Private Data
//- Reference to mesh //- Reference to mesh
const polyMesh& mesh_; const polyMesh& mesh_;

View File

@ -32,7 +32,7 @@ Description
Some common dictionary properties: Some common dictionary properties:
\table \table
Property | Description | Required | Default Property | Description | Required | Default
isoAlgorithm | (cell/topo/point) | no | point isoAlgorithm | (cell/topo/point) | no | topo
regularise | point snapping (bool or enum) | no | true regularise | point snapping (bool or enum) | no | true
\endtable \endtable