Repository navigation
Petsc operators fixes - #3522
dschwoerer wants to merge 4 commits into
Conversation
|
For some reason when I run with this it spams the output with And it does not seem to run aswell. |
Is coming from YBoundary being re-created all over again. Is that maybe from What is the error? Same as before? |
When i run with the current BOUT++ commit in Hermes-3, it does not spam this.
Same error as before |
|
I happened to see this issue/PR and wanted to mention there is also a related issue with using the new petsc operator for thermal conduction with hermes3 (with use_petsc_operator=true). On top of your change in petsc_operators.hxx which changed PETSC_DECIDE there I also had to change PETSC_DECIDE to n_local on line 453 in petsc_operators.cxx: from BOUT_DO_PETSC(MatSetSizes(*temp, n_local, PETSC_DECIDE, n_global, n_global)); to BOUT_DO_PETSC(MatSetSizes(*temp, n_local, n_local, n_global, n_global)); |
|
Thanks a lot, that looks indeed like it needs to be fixed. @totork does it work now?
Like the current bout++/next? Or what ever hermes-3 currently tracks in master? |
Co-authored-by: Stefan Tirkas <tirkas1@llnl.gov>
8912e66 to
94c0462
Compare
|
Seems to work :) |
Fixes: #3517
With this, at least for 1D meshes, the x parallelization works.
@totork Can you verify for fully 3D?