bugpyFormex - Bugs: bug #36692, mesh smoothing

 
 

bug #36692: mesh smoothing

Submitted by:  Sofie Van Cauter <svcauter>
Submitted on:  Wed 20 Jun 2012 08:44:46 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 20 Jun 2012 01:44:30 PM UTC, comment #2:

Trisurface.smoothLowPass is currently not really redundant as it allows the user to include neighbourhood points in one iteration. This should be relatively easy to implement in Mesh.smooth though.

As for the smoothLaplaceHC algorithm, I would have to check how topologies other than tri3 react to the algorithm before adding it to mesh.smooth.

Anyway, we should indeed try to remove TriSurface.smooth by including all functionalities in Mesh.smooth. It's probably not so difficlut to achieve; I'll take a look at it soon.

Tomas Praet <tpraet>
Wed 20 Jun 2012 01:21:25 PM UTC, comment #1:

I think we should see the lambda and mu as two steps of a single iteration. But it should be documented more precisely. If both steps were counted, then an odd number would result in ending with a lambda, while an even number would end with a mu, which may be confusing. So I suggest to keep the number of iterations as it is, but document clearly that k>0 includes an extra smoothing.

The second part: if smoothLaplaceHC works with any mesh, not just triangles, it should indeed be moved to mesh. But if it does not provide anything new/better over Mesh.smooth (resultwise or speedwise), we can maybe just leave it out. Coordinate with Tomas for checking this, and go ahead. And gtssmooth can be left out as well.

Benedict Verhegghe <bverheg>
Project Administrator
Wed 20 Jun 2012 08:44:46 AM UTC, original submission:

The mesh.smooth function currently runs twice the number of iterations than asked by the user. The 'lambda' as well as 'mu' step result in smoothing of the mesh vertices. I suggest we correct this.

The Trisurface.smoothLowPass seems redundant as the same operation can be performed using mesh.smooth. Maybe we can also add the smoothLaplaceHC to mesh.py and remove the trisurface smoothing functions?

Sofie Van Cauter <svcauter>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by tpraet (Posted a comment)
  • -unavailable- added by bverheg (Posted a comment)
  • -unavailable- added by svcauter (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item

    Back to the top


    Powered by Savane 3.1-cleanup1