Bug: noise-plus utility

The noise utility exports or reads in data files to ensight format.

The data could be read in paraview but not in EnSight be cause ensight reader can't support special characters

e.g. reader would skip % sign when reading motorBike case

patch_motorBike_rider-helmet%65.0000.mesh is interpreted as

patch_motorBike_rider-helmet065.0000.mesh

Please update the writer to skip special characters in family name!

Probably this needs to be checked for ensight export in general (?)

@Mattijs

Also details on EP#164

mentioned in issue #132 (closed)

Reassigned to @mark

mentioned in commit 67f6a6cfd5982e811d488e3d68d8c678fe69ff87

Bug was reproducible for motorBike case with following FO surfaceSampling

With latest commit in develop branch, the bug seem to be fixed. (It also changed the folder name!)

master branch output

postProcessing/surfaceSampling/motorBike_rider-helmet%64/
 |-motorBike_rider-helmet%64.case
 |-motorBike_rider-helmet%64.0000.p
 |-motorBike_rider-helmet%64.0000.mesh
 |-fieldsDict

develop branch output

postProcessing/surfaceSampling/motorBike_rider-helmet64/
 |-motorBike_rider-helmet64.case
 |-motorBike_rider-helmet64.0000.p
 |-motorBike_rider-helmet64.0000.mesh
 |-fieldsDict

Status changed to closed

The change in folder-name is mostly for overall consistency. The '%' must be removed from the variable and mesh names. It would be permissible for it to remain in the case name and in the directory name itself (since this is not explicitly mentioned in the case file). However, it is slightly cleaner to have consistent names throughout. In a future modification it would be desirable to shorten the variable and mesh names too (see output from foamToEnsightParts), for several reasons: the names become very long and unreadable; the directory looks very full with multiple variables; it is difficult to selective delete particular time-sets at a later date.

Agree.

In the case that this behaviour (consistent names) is not desired, the line to change:

Foam::fileName Foam::ensightSurfaceWriter::writeCollated
...
<  const fileName baseDir = outputDir.path()/surfName;
>  const fileName baseDir = outputDir.path()/surfaceName;

But I would still retain the cleaned version for the case file.

mentioned in commit 8511117bed88f003ae603871b37958878fc5768e