HarmonicSmoothing::move is broken

Bug #1047641 reported by Garth Wells on 2012-09-07
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
DOLFIN
Medium
Jan Blechta

Bug Description

The function HarmonicSmoothing::move is broken because somewhere makes an assumption on the dof ordering.

Related branches

Changed in dolfin:
assignee: nobody → Anders Logg (logg)
description: updated
Yaakoub El Khamra (yelkhamra) wrote :

With this bug unresolved I cannot use my 2D domain with a hole in it code. It does a snap_boundary same as in the smoothing example. Should I just go with mesh generation, an external mesh generator or are there alternatives? Any suggestions?

Thanks in advance
Regards
Yaakoub

Anders Logg (logg) wrote :

I have made a temporary fix so that an error is only thrown when the parameter "reorder_dofs_serial" is set to True. If set to False, the function works fine.

Changed in dolfin:
status: New → Confirmed
importance: Undecided → Critical
importance: Critical → Medium
Jan Blechta (blechta) on 2013-03-01
Changed in dolfin:
assignee: Anders Logg (logg) → Jan Blechta (blechta)
status: Confirmed → In Progress
Jan Blechta (blechta) on 2013-03-06
Changed in dolfin:
status: In Progress → Fix Committed
Johannes Ring (johannr) wrote :

The submesh demo is disabled because of this bug. Should it be enabled again now that this bug has been fixed? I tried to run it but there were lots of nan's in the smoothed mesh, which is the same as reported in bug #1143977.

On Tue, 12 Mar 2013 13:54:43 -0000
Johannes Ring <email address hidden> wrote:
> The submesh demo is disabled because of this bug. Should it be enabled
> again now that this bug has been fixed? I tried to run it but there
> were lots of nan's in the smoothed mesh, which is the same as
> reported in bug #1143977.
>

That's different functionality: HarmonicSmoothing vs MeshSmoothing.
Just remove fluid_mesh.smooth() in demo. Still fluid_mesh is smoothed by
HarmonicSmoothing on line fluid_mesh.move(structure_mesh).

Jan

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers