Merge lp:~asc/fluidity/solver_reordering_remove into lp:fluidity

Proposed by Adam Candy
Status: Merged
Merged at revision: 3900
Proposed branch: lp:~asc/fluidity/solver_reordering_remove
Merge into: lp:fluidity
Diff against target: 109 lines (+28/-49)
3 files modified
femtools/Solvers.F90 (+4/-1)
schemas/solvers.rnc (+11/-20)
schemas/solvers.rng (+13/-28)
To merge this branch: bzr merge lp:~asc/fluidity/solver_reordering_remove
Reviewer Review Type Date Requested Status
Stephan Kramer Approve
Review via email: mp+89023@code.launchpad.net

Description of the change

Removal of the named solver reordering options.
Notes have been added to both the schema and Solvers.F90 stating the reordering mechanisms have not been tested, and how to use them if you want to try them out.

To post a comment you must log in.
Revision history for this message
Stephan Kramer (s-kramer) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'femtools/Solvers.F90'
--- femtools/Solvers.F90 2011-10-26 14:37:06 +0000
+++ femtools/Solvers.F90 2012-01-18 12:53:28 +0000
@@ -774,7 +774,10 @@
774 end if774 end if
775 775
776 ewrite(1, *) 'Assembling matrix.'776 ewrite(1, *) 'Assembling matrix.'
777 777
778 ! Note the explicitly-described options rcm, 1wd and natural are now not
779 ! listed explicitly in the schema (but can still be used by adding the
780 ! appropriate string in the solver reordering node).
778 call PetscOptionsGetString("", "-ordering_type", ordering_type, &781 call PetscOptionsGetString("", "-ordering_type", ordering_type, &
779 use_reordering, ierr)782 use_reordering, ierr)
780 if (.not. use_reordering) then783 if (.not. use_reordering) then
781784
=== modified file 'schemas/solvers.rnc'
--- schemas/solvers.rnc 2010-10-27 12:18:35 +0000
+++ schemas/solvers.rnc 2012-01-18 12:53:28 +0000
@@ -427,26 +427,17 @@
427 element cache_solver_context {427 element cache_solver_context {
428 empty428 empty
429 }?,429 }?,
430 (430 ## Specify a reordering mechanism supported by PETSc
431 ## Use Reverse Cuthill-McKee reordering to improve cache performance431 ## to improve cache performance, see
432 element reordering {432 ## http://www-unix.mcs.anl.gov/petsc/petsc-as/snapshots/petsc-current/docs/manualpages/MatOrderings/MatGetOrdering.html
433 attribute name { "rcm" }433 ##
434 }|434 ## Note that the use of reordering mechanisms for solves is
435 ## Use one way dissecting reordering to improve cache performance435 ## currently untested. Possible schemes include:
436 element reordering {436 ## Reverse Cuthill-McKee reordering (rcm), a one way dissecting (1wd)
437 attribute name { "1wd" }437 ## algorithm, and also a natural ordering (natural) for debugging.
438 }|438 element reordering {
439 ## Use natural ordering as reordering, i.e. change nothing.439 attribute name { xsd:string }
440 ## Only useful for debugging purposes.440 }?,
441 element reordering {
442 attribute name { "natural" }
443 }|
444 ## Specify any other reordering mechanism supported by PETSc
445 ## see http://www-unix.mcs.anl.gov/petsc/petsc-as/snapshots/petsc-current/docs/manualpages/MatOrderings/MatGetOrdering.html
446 element reordering {
447 attribute name { xsd:string }
448 }
449 )?,
450 ## Extra diagnostics to help debug solver problems441 ## Extra diagnostics to help debug solver problems
451 element diagnostics {442 element diagnostics {
452 ## Print out the norm of vectors and matrices before the443 ## Print out the norm of vectors and matrices before the
453444
=== modified file 'schemas/solvers.rng'
--- schemas/solvers.rng 2010-10-27 12:18:35 +0000
+++ schemas/solvers.rng 2012-01-18 12:53:28 +0000
@@ -448,34 +448,19 @@
448 </element>448 </element>
449 </optional>449 </optional>
450 <optional>450 <optional>
451 <choice>451 <element name="reordering">
452 <element name="reordering">452 <a:documentation>Specify a reordering mechanism supported by PETSc
453 <a:documentation>Use Reverse Cuthill-McKee reordering to improve cache performance</a:documentation>453to improve cache performance, see
454 <attribute name="name">454http://www-unix.mcs.anl.gov/petsc/petsc-as/snapshots/petsc-current/docs/manualpages/MatOrderings/MatGetOrdering.html
455 <value>rcm</value>455
456 </attribute>456Note that the use of reordering mechanisms for solves is
457 </element>457currently untested. Possible schemes include:
458 <element name="reordering">458Reverse Cuthill-McKee reordering (rcm), a one way dissecting (1wd)
459 <a:documentation>Use one way dissecting reordering to improve cache performance</a:documentation>459algorithm, and also a natural ordering (natural) for debugging.</a:documentation>
460 <attribute name="name">460 <attribute name="name">
461 <value>1wd</value>461 <data type="string"/>
462 </attribute>462 </attribute>
463 </element>463 </element>
464 <element name="reordering">
465 <a:documentation>Use natural ordering as reordering, i.e. change nothing.
466Only useful for debugging purposes.</a:documentation>
467 <attribute name="name">
468 <value>natural</value>
469 </attribute>
470 </element>
471 <element name="reordering">
472 <a:documentation>Specify any other reordering mechanism supported by PETSc
473see http://www-unix.mcs.anl.gov/petsc/petsc-as/snapshots/petsc-current/docs/manualpages/MatOrderings/MatGetOrdering.html</a:documentation>
474 <attribute name="name">
475 <data type="string"/>
476 </attribute>
477 </element>
478 </choice>
479 </optional>464 </optional>
480 <element name="diagnostics">465 <element name="diagnostics">
481 <a:documentation>Extra diagnostics to help debug solver problems</a:documentation>466 <a:documentation>Extra diagnostics to help debug solver problems</a:documentation>