← Back to team overview

dolfin team mailing list archive

Re: NSE solver now converges for test problem. Needs more testing but is ok for new release. [...]

 

> Quoting Johan Hoffman <jhoffman@xxxxxxxxxxx>:
>
>>
>> > On Tue, Dec 13, 2005 at 11:23:02PM +0100, dolfin@xxxxxxxxxx wrote:
>> >> Commit from hoffman (2005-12-13 23:22 CET)
>> >> -------------------
>> >>
>> >> NSE solver now converges for test problem. Needs more testing but is
>> ok
>> >> for new release.
>
> Have the results been looked at? I tried it last night and it converged,
> although the results looked very strange.
>
> Garth

I worked on my laptop last night, where I did not have Paraview avaliable,
so I have not checked the results yet. If it is not ok, we should disable
the NSE solver until we've fixed it. I'll try to have another look at it
as soon as I can. If you want to release without it we can do that, and
release again when the NSE solver is avaliable.

/Johan





>
>> >
>> > Great. I will make a new release sometime tomorrow if no one objects
>> > before then.
>>
>> Ok.
>>
>> >> Had to add unistd.h in 2 files (common/meminfo.cpp and
>> >> log/CursesLogger.cpp) to compile.
>> >
>> > ok, I will take a look at this. meminfo.cpp can be removed. But why
>> > did you need it in CursesLogger.cpp? What error message do you get if
>> > you remove unistd.h from that file? It works for me without it so I
>> > can't check.
>>
>> It complained on the function alarm(1) (row 37).
>>
>> /Johan
>>
>>
>>
>>
>> _______________________________________________
>> DOLFIN-dev mailing list
>> DOLFIN-dev@xxxxxxxxxx
>> http://www.fenics.org/cgi-bin/mailman/listinfo/dolfin-dev
>>
>






Follow ups

References