Using the setting "minimal number of cells across" >0 will partially override rules

[wyldckat]

Original Report:

ID:	Category:	Severity:	Reproducibility:	Date Submitted:	Last Update:
13	[enGrid] bug	major	always	2011-09-08 09:29	2012-03-30 10:33
Reporter:	wyldckat	Platform:
Assigned To:	ogloth	OS:
Priority:	normal	OS Version:
Status:	assigned	Product Version:	1.3beta
Product Build:		Resolution:	open
Projection:	none
ETA:	none
Summary:	Using the setting "minimal number of cells across" >0 will partially override rules
Description:	This was detected in bug 0000009, but this seems to be a global bug, not just related to the version for Windows. Basically, if we follow the Damper Duct tutorial until the end of the section"http://engits.eu/wiki/index.php/Tutorials/Version1.3/DamperDuct#Improve_Damper_Edges", [^] while using the global setting "minimal number of cells across" with a value of 1, the defined rules in that tutorial section will not be properly followed. Proof is in the attached file "enGrid 1.3_beta - rules not followed.png". When the aforementioned value is 0, then the image looks like the one in the tutorial: "enGrid 1.3_beta - rules followed.png"
Steps To Reproduce:
Additional Information:
Attached Files:	enGrid 1.3_beta - rules not followed.png (233 KB) 2011-09-08 09:29 enGrid 1.3_beta - rules followed.png (219 KB) 2011-09-08 09:29

[wyldckat]

Notes
(0000036)
wyldckat
2011-09-08 09:32	As a result of the debugging process undertaken for the bug report 0000009, the incident was tracked down to the following for loop: for (int i_level = 0; i_level < 3*m_MinMumCellsAcross; ++i_level) {  foreach (vtkIdType id_node, local_nodes) {    for (int i = 0; i < m_Part.n2nGSize(id_node); ++i) {     local_nodes.insert(m_Part.n2nGG(id_node, i));    }   } } It's located in "libengrid/updatedesiredmeshdensity.cpp", method "void UpdateDesiredMeshDensity::operate()", a bit after the comment "// cells across branches". The defined rules should be taken into account in this loop.
(0000068)
ogloth
2012-03-30 10:33	I will add this to my things to do ;-) The minimal number of cells across should really be part of the mesh settings dialogue, rather than a global option.

[wyldckat]

Old report 0000009 is now Issue #17.

[wyldckat]

Attachments now located at https://gist.github.com/3071978 :

• https://raw.github.com/gist/3071978/ed23cc255e89d4d5520f4ea6f736e16f6cd138c7/enGrid+1.3_beta+-+rules+followed.png
• https://raw.github.com/gist/3071978/f1abb8048bf7bdbfc03b8d6a7d1e714135f3af6f/enGrid+1.3_beta+-+rules+not+followed.png

[ogloth]

This feature has moved to the mesh settings dialogue. The global setting has no effect now.