View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001931 | OpenFOAM | Bug | public | 2015-11-30 18:34 | 2016-02-02 18:29 |
Reporter | Assigned To | henry | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | unable to reproduce | ||
Platform | GNU/Linux | OS | Ubuntu | OS Version | 12.04 |
Summary | 0001931: circleSet sampling spams errors on decomposed cases | ||||
Description | circleSets::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 Reproduce | 1 - Add a circleSet sampling to a case 2 - Decompose the case and run | ||||
Tags | No tags attached. | ||||
|
Do you have a patch you can provide to restrict the warning until the points on all the processes are collected? |
|
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? |
|
@esteldunedain: Can you please provide a test case? It would speed things up on the development side and respective testing, including for #1932 :) |
|
@wyldckat, I'll upload a test case asap (which may be tomorrow). |
|
Orphaned report. |
Date Modified | Username | Field | Change |
---|---|---|---|
2015-11-30 18:34 |
|
New Issue | |
2015-11-30 18:36 | henry | Note Added: 0005693 | |
2015-11-30 18:58 |
|
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 |
|
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 |