Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Non-working CGNR iterative solver in OpenCL mode #198

Closed
GoogleCodeExporter opened this issue Aug 12, 2015 · 2 comments
Closed

Non-working CGNR iterative solver in OpenCL mode #198

GoogleCodeExporter opened this issue Aug 12, 2015 · 2 comments
Assignees
@myurkin
Labels
accuracy Improves simulation accuracy bug Incorrect code behavior comp-Logic Related to internal code logic OpenCL Running on GPUs and similar devices pri-Medium Worth assigning to a milestone
Milestone
1.4

Comments

@GoogleCodeExporter
Copy link 

There was an error in conjugate-transpose part (her=true) of matvec product for 
a long time (at least from release 1.1). This part is used only in CGNR 
iterative solver. Moreover, it was negligible for a sphere, so a different 
shape (with different volume fraction of a computational grid) is required to 
notice it (that's why it was not noticed by automatic tests).

For example, simple test is 'adda_ocl.exe -iter cgnr -shape box', which causes 
convergence failure.

Original issue reported on code.google.com by yurkin on 28 May 2014 at 11:42
@GoogleCodeExporter
Copy link
Author 

Fixed by r1345.

r1345 - ac2582e

Original comment by yurkin on 28 May 2014 at 11:43

  • Changed state: Fixed

@GoogleCodeExporter GoogleCodeExporter added auto-migrated accuracy Improves simulation accuracy comp-Logic Related to internal code logic pri-Medium Worth assigning to a milestone OpenCL Running on GPUs and similar devices labels Aug 12, 2015
@myurkin myurkin added bug Incorrect code behavior and removed auto-migrated labels Aug 13, 2015
@myurkin myurkin added this to the 1.3 milestone Aug 14, 2015
@myurkin myurkin self-assigned this Aug 14, 2015
@myurkin
Copy link
Member

myurkin commented Aug 14, 2015

/cc @mapclyps

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@myurkin myurkin

Labels
accuracy Improves simulation accuracy bug Incorrect code behavior comp-Logic Related to internal code logic OpenCL Running on GPUs and similar devices pri-Medium Worth assigning to a milestone
Projects
None yet
Milestone
1.4
Development

No branches or pull requests
2 participants
@myurkin @GoogleCodeExporter