View Issue Details

IDProjectCategoryView StatusLast Update
0003763OpenFOAMFeaturepublic2021-12-10 09:50
Reportertniemi Assigned Towill  
PrioritynormalSeverityminorReproducibilityN/A
Status resolvedResolutionfixed 
Product Versiondev 
Fixed in Versiondev 
Summary0003763: Enable binary support for VTK set writer
DescriptionFollowing the recent changes to streamlines, adding binary support for VTK streamlines should now be trivial. Here is a simple patch where false -> writeFormat_ so that user can set it to binary if needed.

Binary streamlines save a bit of space if there are many lines and/or long streams with multiple variables.
TagsNo tags attached.

Activities

tniemi

2021-12-08 15:05

reporter  

patch.diff (492 bytes)   
diff --git a/src/sampling/sampledSet/writers/vtk/vtkSetWriter.C b/src/sampling/sampledSet/writers/vtk/vtkSetWriter.C
index d5ac3971c..17b88c2ee 100644
--- a/src/sampling/sampledSet/writers/vtk/vtkSetWriter.C
+++ b/src/sampling/sampledSet/writers/vtk/vtkSetWriter.C
@@ -79,7 +79,7 @@ void Foam::vtkSetWriter::write
     (
         outputDir/setName + ".vtk",
         setName,
-        false,
+        writeFormat_,
         set.pointCoords()(),
         set.vertices(),
         set.lines(),
patch.diff (492 bytes)   

will

2021-12-10 09:50

manager   ~0012307

Thanks for pointing this out. Easy fix.

https://github.com/OpenFOAM/OpenFOAM-dev/commit/d8a56df156e647b1d05d09b61e49e82e7b641e4b

Issue History

Date Modified Username Field Change
2021-12-08 15:05 tniemi New Issue
2021-12-08 15:05 tniemi File Added: patch.diff
2021-12-10 09:50 will Assigned To => will
2021-12-10 09:50 will Status new => resolved
2021-12-10 09:50 will Resolution open => fixed
2021-12-10 09:50 will Fixed in Version => dev
2021-12-10 09:50 will Note Added: 0012307