View Issue Details

IDProjectCategoryView StatusLast Update
0001931OpenFOAMBugpublic2016-02-02 18:29
Reporteruser763Assigned Tohenry  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionunable to reproduce 
PlatformGNU/LinuxOSUbuntuOS Version12.04
Summary0001931: circleSet sampling spams errors on decomposed cases
DescriptioncircleSets::calcSamples method spams numerous errors like this on decomposed cases:

--> FOAM Warning :
    From function void circleSet::calcSamples(DynamicList<point>&, DynamicList<label>&, DynamicList<label>&, DynamicList<label>&, DynamicList<scalar>&) const
    in file sampledSet/circle/circleSet.C at line 83
    Unable to find cell at point id 0 at location (19.1889 12.5028 -3.5)

As far as I understand, the cause is that each process searches on its subdomain for all the points, and prints a warning for each one for which a corresponding cell isn't found.

Despite the warning, the sampling works as intended, given that the points found on all proccesses are combined later on the sampledSets::combineSampledSets method.

Warning messages for should probably not be shown, at least until the points on all the processes are collected.

Note: Tested on 2.3.x, but the relevant source code seems identical on 2.4 and 3.0, so I assume it's unfixed
Steps To Reproduce1 - Add a circleSet sampling to a case
2 - Decompose the case and run
TagsNo tags attached.

Relationships

related to 0001932 resolvedhenry circleSet sampling points are unordered on decomposed cases 

Activities

henry

2015-11-30 18:36

manager   ~0005693

Do you have a patch you can provide to restrict the warning until the points on all the processes are collected?

user763

2015-11-30 18:58

  ~0005694

Sorry, not really. I can't think of any easy way of doing that without some fairly extensive rewritting of the circleSet class. Maybe just ignoring the warnings on parallel cases?

wyldckat

2015-11-30 19:34

updater   ~0005695

@esteldunedain: Can you please provide a test case? It would speed things up on the development side and respective testing, including for #1932 :)

user763

2015-11-30 19:46

  ~0005697

@wyldckat, I'll upload a test case asap (which may be tomorrow).

henry

2016-02-02 18:29

manager   ~0005877

Orphaned report.

Issue History

Date Modified Username Field Change
2015-11-30 18:34 user763 New Issue
2015-11-30 18:36 henry Note Added: 0005693
2015-11-30 18:58 user763 Note Added: 0005694
2015-11-30 19:34 wyldckat Note Added: 0005695
2015-11-30 19:34 wyldckat Relationship added related to 0001932
2015-11-30 19:46 user763 Note Added: 0005697
2016-02-02 18:29 henry Note Added: 0005877
2016-02-02 18:29 henry Status new => closed
2016-02-02 18:29 henry Assigned To => henry
2016-02-02 18:29 henry Resolution open => unable to reproduce