Martin Sandve Alnæs wrote:
> On Mon, Dec 22, 2008 at 1:02 PM, Garth N. Wells <[email protected]> wrote:
>>
>> Martin Sandve Alnæs wrote:
>>> This check isn't general either.
>>>
>> Can you give me an example of when it fails?
> 
> It will fail for symmetric tensor elements.
>

Which we don't yet support :).

>>>   // Test for cell-based element type
>>> -  const uint tdim = mesh.topology().dim();
>>> -  bool only_cell_dofs = dofmap.needs_mesh_entities(tdim);
>>> -  for (uint i = 0; i < tdim; i++)
>>> -  {
>>> -    if (dofmap.needs_mesh_entities(i))
>>> -      only_cell_dofs = false;
>>> -  }
>>> -  if (only_cell_dofs)
>>> +  if( dofmap.local_dimension() == std::pow(mesh.topology().dim(), rank) )
>>>     data_type = "cell";
>>>
>>>
>>> This is the check I suggested to Anders earlier:
>>>
>>> const uint tdim = mesh.topology().dim();
>>> if(dofmap.num_entity_dofs(tdim) == dofmap.local_dimension())
>>>
>> What does num_entity_dofs(tdim) return?
>>
>> Garth
> 
> The number of dofs associated with a cell (mesh entity of dimension tdim).
> dofmap.num_entity_dofs(0) would return the number of dofs associated
> with the vertices.

OK. That would make the test easy.

> 
>>> Or hasn't FFC implemented UFC 1.1 yet?

FFC hasn't implemented num_entity_dofs(tdim)

Garth

>>>
>>> Martin
>>>
>>>
>>>
>>> On Mon, Dec 22, 2008 at 12:42 PM, DOLFIN <[email protected]> wrote:
>>>> One or more new changesets pushed to the primary dolfin repository.
>>>> A short summary of the last three changesets is included below.
>>>>
>>>> changeset:   5434:d8bb5b439227b34030db2c7682da63a1841924c5
>>>> tag:         tip
>>>> user:        "Garth N. Wells <[email protected]>"
>>>> date:        Mon Dec 22 11:41:53 2008 +0000
>>>> files:       dolfin/io/VTKFile.cpp
>>>> description:
>>>> Change test for cell-based data in VTKFile.
>>>>
>>>> The previous test failed for DG elements of order k > 0.
>>>>
>>>>
>>>> changeset:   5433:b949069baa57b73ac38b88f4b2de1c952c602743
>>>> user:        Johannes Ring <[email protected]>
>>>> date:        Mon Dec 22 10:30:47 2008 +0100
>>>> files:       scons/simula-scons/simula_scons/pkgconfiggenerators/gmp.py
>>>> description:
>>>> simula-scons update (added missing cstdlib in GMP test).
>>>>
>>>>
>>>> changeset:   5432:72d035bcc3c234475057ca8f8dfed7cceb5352b7
>>>> user:        Anders Logg <[email protected]>
>>>> date:        Sun Dec 21 23:27:33 2008 +0100
>>>> files:       dolfin/function/FunctionSpace.cpp dolfin/main/MPI.cpp
>>>> description:
>>>> New attempt to fix bug...
>>>>
>>>> ----------------------------------------------------------------------
>>>> For more details, visit http://www.fenics.org/hg/dolfin
>>>> _______________________________________________
>>>> DOLFIN-dev mailing list
>>>> [email protected]
>>>> http://www.fenics.org/mailman/listinfo/dolfin-dev
>>>>
>>> _______________________________________________
>>> DOLFIN-dev mailing list
>>> [email protected]
>>> http://www.fenics.org/mailman/listinfo/dolfin-dev


_______________________________________________
DOLFIN-dev mailing list
[email protected]
http://www.fenics.org/mailman/listinfo/dolfin-dev

Reply via email to