Typo in solve arguments in Chapter 1

Bug #931284 reported by Anders Logg on 2012-02-13
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
FEniCS Book Project
Confirmed
Medium
Unassigned

Bug Description

Reported by Fredrik Valdmanis:

=== modified file 'tex/1.tex'
--- tex/1.tex 2011-11-17 12:37:20 +0000
+++ tex/1.tex 2012-02-10 13:09:41 +0000
@@ -671,7 +671,7 @@
 LU factorization (ILU) is a popular and
 robust all-round preconditioner, so let us try the CG--ILU pair:
 \begin{python}
-solve(a == L, u, bc)
+solve(a == L, u, bc,
       solver_parameters={"linear_solver": "cg",
                          "preconditioner": "ilu"})

Anders Logg (logg) on 2012-02-13
Changed in fenics-book:
status: New → Confirmed
importance: Undecided → Medium

On 13 February 2012 07:46, Anders Logg <email address hidden> wrote:
> Public bug reported:
>
> Reported by Fredrik Valdmanis:
>
> === modified file 'tex/1.tex'
> --- tex/1.tex   2011-11-17 12:37:20 +0000
> +++ tex/1.tex   2012-02-10 13:09:41 +0000
> @@ -671,7 +671,7 @@
>  LU factorization (ILU) is a popular and
>  robust all-round preconditioner, so let us try the CG--ILU pair:
>  \begin{python}
> -solve(a == L, u, bc)
> +solve(a == L, u, bc,
>       solver_parameters={"linear_solver": "cg",
>                          "preconditioner": "ilu"})
>

Will this work? Is there are guarantee that solve(a == L, bc) will use
symmetric application of the bcs?

Garth

> ** Affects: fenics-book
>     Importance: Medium
>         Status: Confirmed
>
> ** Changed in: fenics-book
>       Status: New => Confirmed
>
> ** Changed in: fenics-book
>   Importance: Undecided => Medium
>
> --
> You received this bug notification because you are a member of FEniCS
> Book Editors, which is subscribed to FEniCS Book Project.
> https://bugs.launchpad.net/bugs/931284
>
> Title:
>  Typo in solve arguments in Chapter 1
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/fenics-book/+bug/931284/+subscriptions

Anders Logg (logg) wrote :

On Mon, Feb 13, 2012 at 07:57:56AM -0000, Garth Wells wrote:
> On 13 February 2012 07:46, Anders Logg <email address hidden> wrote:
> > Public bug reported:
> >
> > Reported by Fredrik Valdmanis:
> >
> > === modified file 'tex/1.tex'
> > --- tex/1.tex   2011-11-17 12:37:20 +0000
> > +++ tex/1.tex   2012-02-10 13:09:41 +0000
> > @@ -671,7 +671,7 @@
> >  LU factorization (ILU) is a popular and
> >  robust all-round preconditioner, so let us try the CG--ILU pair:
> >  \begin{python}
> > -solve(a == L, u, bc)
> > +solve(a == L, u, bc,
> >       solver_parameters={"linear_solver": "cg",
> >                          "preconditioner": "ilu"})
> >
>
> Will this work? Is there are guarantee that solve(a == L, bc) will
> use symmetric application of the bcs?

The solver will only do a symmetric application if the "symmetric"
parameter is set, so it should be

 solve(a == L, u, bc,
       solver_parameters={"symmetric": True,
                          "linear_solver": "cg",
                          "preconditioner": "ilu"})

--
Anders

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers