Also to be 100% clear, I'm not asking you to refactor this code, I'm just
pointing out an opportunity, volunteering to make the change, and asking
for your approval :)


On Thu, Feb 6, 2014 at 10:46 AM, Aron Ahmadia <aron@ahmadia.net> wrote:

> Is there a reason to copy and paste the template, or should we be reusing
> the test_poisson.py file as a module?
>
>
> On Thu, Feb 6, 2014 at 10:44 AM, mfarthin <notifications@github.com>wrote:
>
>> In test/poisson_2d/test_poisson2d.py:
>>
>> > +    opts.gatherArchive=True
>> > +    nList[0].femSpaces[0]  = default_n.C0_AffineQuadraticOnSimplexWithNodalBasis
>> > +    nList[0].linearSolver=default_n.KSP_petsc4py
>> > +    nList[0].multilevelLinearSolver=default_n.KSP_petsc4py
>> > +    #set ksp options
>> > +    from petsc4py import PETSc
>> > +    OptDB = PETSc.Options()
>> > +    OptDB.setValue('ksp_type','preonly')
>> > +    OptDB.setValue('pc_type','lu')
>> > +    OptDB.setValue('pc_factor_mat_solver_package','superlu_dist')
>> > +    nList[0].numericalFluxType = default_n.Advection_DiagonalUpwind_Diffusion_SIPG_exterior
>> > +    #nList[0].linearSolver=default_n.LU
>> > +    #nList[0].multilevelLinearSolver=default_n.LU
>> > +    ns = NumericalSolution.NS_base(so,pList,nList,so.sList,opts)
>> > +    ns.calculateSolution('poisson_2d_c0p2')
>> > +    assert(True)
>>
>> Yeah, I was following the template in test_poisson.py. I think it was
>> there as a reminder that we could add checking an actual condition in the
>> test.
>>
>> --
>> Reply to this email directly or view it on GitHub<https://github.com/erdc-cm/proteus/pull/38/files#r9504473>
>> .
>>
>
>