Re: [Libmesh-users] fe szabab

2016-10-28 Thread John Peterson
On Fri, Oct 28, 2016 at 1:16 PM, Manav Bhatia  wrote:

> Hi,
>
> https://github.com/libMesh/libmesh/blob/master/src/fe/fe_
> szabab_shape_2D.C#L776  libmesh/blob/master/src/fe/fe_szabab_shape_2D.C#L776>  : It should also
> list TRI3.
>
> Similarly,
>
> https://github.com/libMesh/libmesh/blob/master/src/fe/fe_
> szabab_shape_2D.C#L814  libmesh/blob/master/src/fe/fe_szabab_shape_2D.C#L814>  should also list
> QUAD4.
>

OK, seems like we should also add

INSTANTIATE_FETEST(SECOND, SZABAB, TRI3);
INSTANTIATE_FETEST(SECOND, SZABAB, QUAD4);

to fe_szabab_test.C unit test file in that case.  I can take a stab at
this..

-- 
John
--
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive. 
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
___
Libmesh-users mailing list
Libmesh-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libmesh-users


[Libmesh-users] fe szabab

2016-10-28 Thread Manav Bhatia
Hi, 

https://github.com/libMesh/libmesh/blob/master/src/fe/fe_szabab_shape_2D.C#L776 

  : It should also list TRI3.

Similarly, 

https://github.com/libMesh/libmesh/blob/master/src/fe/fe_szabab_shape_2D.C#L814 

  should also list QUAD4. 

-Manav
--
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive. 
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
___
Libmesh-users mailing list
Libmesh-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libmesh-users