[Mars409]

foamFormatConvert is an OpenFOAM utility to convert a case's simulation results btwn binary and ASCII, depending on what system/controlDict specifies for writeFormat. But in v10 it corrupts the origId and origProcId files.



The problem is fixed by adding a line to call particle::readFields(parcels):

forAll(cloudDirs, i)
            {
                fileName dir(cloud::prefix/cloudDirs[i]);

                Cloud<passiveParticle> parcels(meshPtr(), cloudDirs[i], false);

                particle::readFields(parcels); // [Bug fix] write origId and origProcId

                parcels.writeObject
                (
                    runTime.writeFormat(),
                    IOstream::currentVersion,
                    runTime.writeCompression(),
                    parcels.size()
                );

The cause of the bug is the the constructor of Cloud only picks up the positions file to build a particle list, leaving the origId and origProc lists uninitalized. particle::readFields(parcels) (a static function call) takes care of this.

[olesen]

This has been raised as an issue
https://develop.openfoam.com/Develop.../-/issues/3051
should also be solved now in time for the OpenFOAM-v2312 release.

[olesen]

Originally Posted by Mars409

foamFormatConvert is an OpenFOAM utility to convert a case's simulation results btwn binary and ASCII, depending on what system/controlDict specifies for writeFormat.

The preferred fixed code:

// OLD:     Cloud<passiveParticle> parcels(meshPtr(), cloudDirs[i], false);
// NEW     - with readFields = true
                passiveParticleCloud parcels(meshPtr(), cloudDirs[i], true);

Changing to passiveParticleCloud exposes the readFields directly.