View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003763 | OpenFOAM | Feature | public | 2021-12-08 15:05 | 2021-12-10 09:50 |
| Reporter | tniemi | Assigned To | will | ||
| Priority | normal | Severity | minor | Reproducibility | N/A |
| Status | resolved | Resolution | fixed | ||
| Product Version | dev | ||||
| Fixed in Version | dev | ||||
| Summary | 0003763: Enable binary support for VTK set writer | ||||
| Description | Following 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. | ||||
| Tags | No tags attached. | ||||
|
|
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(),
|
|
|
Thanks for pointing this out. Easy fix. https://github.com/OpenFOAM/OpenFOAM-dev/commit/d8a56df156e647b1d05d09b61e49e82e7b641e4b |
| 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 |
