DOLFIN_NOPLOT not sufficient for general case

Bug #493099 reported by Garth Wells on 2009-12-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
DOLFIN
High
Unassigned

Bug Description

DOLFIN_NOPLOT is used to turn plotting off for the buildbots. However, it only works for simple one lines cases. For example, it does not work for

    p = plot(intersection, rescale=True, wireframe=False, axes=True,scalar_bar=False)
    p.elevate(-50)

If DOLFIN_NOPLOT is set, the first line returns nothing, hence the second line will fail.

I think I remember Hans Petter sent me a fix for this a while back
that I thought would be applied to Viper. It seems it hasn't been yet.

I'll dig up the patch and see if it can be applied.

--
Anders

On Sun, Dec 06, 2009 at 10:32:30AM -0000, Garth Wells wrote:
> Public bug reported:
>
> DOLFIN_NOPLOT is used to turn plotting off for the buildbots. However,
> it only works for simple one lines cases. For example, it does not work
> for
>
> p = plot(intersection, rescale=True, wireframe=False, axes=True,scalar_bar=False)
> p.elevate(-50)
>
> If DOLFIN_NOPLOT is set, the first line returns nothing, hence the
> second line will fail.
>
> ** Affects: dolfin
> Importance: High
> Status: New
>

Anders Logg (logg) wrote :

Fixed using HPL's DoNothing class.

Changed in dolfin:
status: New → Fix Committed
Anders Logg (logg) on 2010-02-03
Changed in dolfin:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers