View Issue Details

IDProjectCategoryView StatusLast Update
0002691OpenFOAM[All Projects] Bugpublic2017-09-09 22:35
Reporterahojukka5Assigned Tohenry 
PrioritynormalSeveritycrashReproducibilityhave not tried
Status resolvedResolutionfixed 
Product Version 
Fixed in Version5.x 
Summary0002691: Unable to convert UNV mesh
DescriptionThis is already reported in issue 0001699. A simple test mesh to reproduce problem is attached. Mesh is generated using newest SALOME. Two mesh files is attached. First one works while other one does not. The only change in SALOME is that in non-working version a face group is created.
TagsNo tags attached.

Relationships

related to 0001699 closedhenry Error IdeasUnvToFoam 
related to 0002627 closedwyldckat Cell type 22 not supported in ideasUnvToFoam.C file 

Activities

ahojukka5

2017-09-07 11:56

reporter  

test_meshes.zip (860 bytes)

wyldckat

2017-09-09 16:47

updater  

ideasUnvToFoam.C (35,764 bytes)

wyldckat

2017-09-09 16:49

updater   ~0008733

@ahojukka5: This is a 2D mesh. The 'working' mesh you've provided is misleading, because it only imported the 4 points that were defined in the UNV file, but the UNV file has zero 3D cells. Therefore this is an invalid mesh either way.

You can find out how to use Salome to generate 3D meshes from 2D meshes here: https://www.cfd-online.com/Forums/openfoam-meshing-other/190522-ideasunvtofoam-cell-type-not-supported.html#post662490 - start reading from post #6.

wyldckat

2017-09-09 16:52

updater   ~0008734

@henry: I've attached the file "ideasUnvToFoam.C", which can be used to update OpenFOAM 4.x, 5.x and dev. The change is shown in the attached file "proposition_v1.patch", namely:

 - When the mesh has 0 cells, give a warning about it and indicate that only 3D meshes are supported.

proposition_v1.patch (776 bytes)
diff --git a/applications/utilities/mesh/conversion/ideasUnvToFoam/ideasUnvToFoam.C b/applications/utilities/mesh/conversion/ideasUnvToFoam/ideasUnvToFoam.C
index c1a3512..6f506e8 100644
--- a/applications/utilities/mesh/conversion/ideasUnvToFoam/ideasUnvToFoam.C
+++ b/applications/utilities/mesh/conversion/ideasUnvToFoam/ideasUnvToFoam.C
@@ -478,6 +478,15 @@ void readCells
 
     Info<< "Read " << cellVerts.size() << " cells"
         << " and " << boundaryFaces.size() << " boundary faces." << endl;
+
+    if (!cellVerts.size())
+    {
+        WarningInFunction
+            << "CAUTION: There are no cells in this mesh!"
+            << " Keep in mind that 2D meshes are not supported, these must"
+            << " always be 3D."
+            << endl;
+    }
 }
 
 
proposition_v1.patch (776 bytes)

henry

2017-09-09 22:35

manager   ~0008739

Thanks Bruno

Resolved in OpenFOAM-5.x by commit e4c205229198c178aae9127e5869dcc9ed5d9c14

Resolved in OpenFOAM-dev by commit 9faf06fe54f9e066af532d3ac6c97981dc27424e

Issue History

Date Modified Username Field Change
2017-09-07 11:56 ahojukka5 New Issue
2017-09-07 11:56 ahojukka5 File Added: test_meshes.zip
2017-09-09 16:17 wyldckat Summary Unable to convert uav mesh => Unable to convert UNV mesh
2017-09-09 16:17 wyldckat Relationship added related to 0001699
2017-09-09 16:47 wyldckat File Added: ideasUnvToFoam.C
2017-09-09 16:49 wyldckat Note Added: 0008733
2017-09-09 16:50 wyldckat Relationship added related to 0002627
2017-09-09 16:52 wyldckat File Added: proposition_v1.patch
2017-09-09 16:52 wyldckat Note Added: 0008734
2017-09-09 16:52 wyldckat Assigned To => henry
2017-09-09 16:52 wyldckat Status new => assigned
2017-09-09 22:35 henry Status assigned => resolved
2017-09-09 22:35 henry Resolution open => fixed
2017-09-09 22:35 henry Fixed in Version => 5.x
2017-09-09 22:35 henry Note Added: 0008739