Skip to content
Snippets Groups Projects

CloudToVTK functionObject broken.

    • Closed Issue created by Admin @OpenFOAM-admin

    Pstream::gatherList is called in CloudToVTK::writeData, which is called only on the master - thereby failing.

    https://develop.openfoam.com/Development/OpenFOAM-plus/blob/975dc1d70217e8aaa4c0a9703da7283a21ec35bb/src/lagrangian/intermediate/submodels/CloudFunctionObjects/CloudToVTK/CloudToVTK.C#L117

    Please find attached a patch to mend this.CloudToVTK.patch

    Edited

      Child items

      0
      Show labels

      No child items are currently assigned. Use child items to break down this issue into smaller parts.

        Activity

        • Admin changed the description

          changed the description

          By Paul Kieckhefen on 2018-08-29T15:28:03 (imported from GitLab project)

        • Prashant Sonakar
          Prashant Sonakar @Prashant ·
          Developer

          @mark

        • Admin
          Admin @OpenFOAM-admin ·
          Author Maintainer

          Also: it seems that perma-disabling binary writing is not neccessary ( binary_=false; can be safely removed).

          By Paul Kieckhefen on 2018-08-30T07:11:54 (imported from GitLab project)

        • Mark OLESEN assigned to @mark

          assigned to @mark

        • Mark OLESEN
          Mark OLESEN @mark ·
          Maintainer

          Hi @pkieck - Thanks for the report. I would actually like to kill off CloudToVTK entirely for several reasons

          • horrible old code,
          • memory inefficient vtk encoding
          • likely breakage for 64-bit labels (see #270 (closed))
          • legacy format only.

          Assuming that you are using 1806, could you try out the vtkCloud function object as an alternative?

          • src/functionObjects/lagrangian/vtkCloud/vtkCloud.H
          • tutorials/lagrangian/coalChemistryFoam/simplifiedSiwek
          • tutorials/lagrangian/reactingParcelFoam/filter

          This provides the XML-based VTP output, more control about which fields to output and also emits a file-series (JSON format) for convenient loading in paraview.

          Edited by Mark OLESEN
        • Admin
          Admin @OpenFOAM-admin ·
          Author Maintainer

          Thank you for the response - vtkCloud looks much better.

          Killing it off would certainly resolve the issue :)

          By Paul Kieckhefen on 2018-09-10T15:36:52 (imported from GitLab project)

        • Mark OLESEN
          Mark OLESEN @mark ·
          Maintainer

          @andy - OK to now kill this off?

        • Admin
          Admin @OpenFOAM-admin ·
          Author Maintainer

          yep

          By Andrew Heather on 2018-09-19T16:12:19 (imported from GitLab project)

        • Mark OLESEN mentioned in commit 71e90eea0e990974fe4695ccb654df3c9df9f3a9

          mentioned in commit 71e90eea0e990974fe4695ccb654df3c9df9f3a9

        • Mark OLESEN closed

          closed

        Please register or sign in to reply