View Issue Details

IDProjectCategoryView StatusLast Update
0002234OpenFOAMPatchpublic2016-09-06 12:58
ReporterMattijsJAssigned Tohenry 
PrioritylowSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformGNU/LinuxOSOpenSuSEOS Version13.2
Product Versiondev 
Fixed in Versiondev 
Summary0002234: algebraicPairGAMGAgglomeration agglomerates based on upper coefficients only
Descriptionit does

agglomerate(mesh, mag(matrix.upper()));

attached a fix for if asymmetric matrices.
TagsNo tags attached.

Activities

MattijsJ

2016-09-04 19:48

reporter  

algebraicPairGAMGAgglomeration.C (2,144 bytes)

henry

2016-09-04 21:13

manager   ~0006822

Why use the max of the upper and lower coefficient?

> max(mag(matrix.upper()), mag(matrix.lower()))

What about using the sum:

mag(matrix.upper()) + mag(matrix.lower())

MattijsJ

2016-09-06 12:46

reporter   ~0006833

From what I can see in the literature the strength of connections is taken as the max, e.g. from wissrech.ins.uni-bonn.de/research/pub/metsch/emg-2005.ps.gz
"Also note that, with this defini-
tion of strength, it is possible that a point i strongly
depends on j, but point j only weakly depends on i,
even though A is symmetric"

henry

2016-09-06 12:58

manager   ~0006835

Resolved by commit 4232f90093b87323bd3d96d9fab7b209f19ed1db

Issue History

Date Modified Username Field Change
2016-09-04 19:48 MattijsJ New Issue
2016-09-04 19:48 MattijsJ File Added: algebraicPairGAMGAgglomeration.C
2016-09-04 21:13 henry Note Added: 0006822
2016-09-06 12:46 MattijsJ Note Added: 0006833
2016-09-06 12:58 henry Assigned To => henry
2016-09-06 12:58 henry Status new => resolved
2016-09-06 12:58 henry Resolution open => fixed
2016-09-06 12:58 henry Fixed in Version => dev
2016-09-06 12:58 henry Note Added: 0006835