minor change to PDESystem.py

Asked by Christopher DeMars on 2012-09-07

In order to get the Lshape demo to run, I needed to slightly modify PDESystem.py:

Line 84 - 85:
        if isinstance(problem, dolfin.cpp.mesh.Mesh):
        #if isinstance(problem, Mesh):
changed to:
        #if isinstance(problem, dolfin.cpp.mesh.Mesh):
        if isinstance(problem, Mesh):

Perhaps there was a change upstream in the dolfin API. Regardless, it works once the line is swapped.

Thanks for such a brilliant extension to FEniCS: solving multiple, linked PDEs should have been a goal of FEniCS from day one. I'll probably have more comments, questions and bug reports as I delve more deeply into the library. Thanks for putting up with my pestering.

Cheers,
--Christopher

Question information

Language:
English Edit question
Status:
Answered
For:
CBC.PDESys Edit question
Assignee:
No assignee Edit question
Last query:
Last reply:

On Friday, 7 September 2012, Christopher DeMars wrote:

> New question #207968 on CBC.PDESys:
> https://answers.launchpad.net/cbcpdesys/+question/207968
>
> In order to get the Lshape demo to run, I needed to slightly modify
> PDESystem.py:
>
> Line 84 - 85:
> if isinstance(problem, dolfin.cpp.mesh.Mesh):
> #if isinstance(problem, Mesh):
> changed to:
> #if isinstance(problem, dolfin.cpp.mesh.Mesh):
> if isinstance(problem, Mesh):
>
> Perhaps there was a change upstream in the dolfin API. Regardless, it
> works once the line is swapped.

There was and I had to make that silly change to make it run. I guess now I
can change it back.

>
> Thanks for such a brilliant extension to FEniCS: solving multiple, linked
> PDEs should have been a goal of FEniCS from day one. I'll probably have
> more comments, questions and bug reports as I delve more deeply into the
> library. Thanks for putting up with my pestering.
>
>
Thank you very much:-) i think so too by the way. The
linear/nonlinearsolver classes in dolfin are simply too simple. Please
don't hesitate to ask and even give your honest opinion. I want to improve
this nice piece of software further, make it more intuitive to use etc. And
I need help to be able to do that.

Best regards

Mikael

> Cheers,
> --Christopher
>
> --
> You received this question notification because you are an answer
> contact for CBC.PDESys.
>

Can you help with this problem?

Provide an answer of your own, or ask Christopher DeMars for more information if necessary.

To post a message you must log in.