DOLFIN_NOPLOT not sufficient for general case

[Garth Wells]

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.

[Anders Logg (logg)]

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)]

Fixed using HPL's DoNothing class.