bugpyFormex - Bugs: bug #43869, TriSurface.centerline is not...

 
 

bug #43869: TriSurface.centerline is not loaded by default when calling the method

Submitted by:  francesco <francio>
Submitted on:  Mon 22 Dec 2014 03:21:50 PM UTC  
 
Category:  Plugins Severity:  2 - Minor
Item Group:  None Status:  In Progress
Privacy:  Public Assigned 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.

 

Sat 10 Jan 2015 08:34:00 AM UTC, comment #3: 

ok but I am not sure how you want to do it. do it for one I i will copy the same strategy also for remesh, though at this point it needs to be done also for gts and all the other functions that require external modules

francesco <francio>
Project Member
Fri 09 Jan 2015 09:37:01 PM UTC, comment #2: 

I reopened the bug because the solution is not a (fully) good one. vmtk is not a required dependency of pyFormex, so its functionality should not be included by default. The proper way would be to test for vmtk and only load the functionality if vmtk is present, and adding (or at least enabling) the vmtk menu item in the surface menu from a function in the vmtk_itf module.

Benedict Verhegghe <bverheg>
Project Administrator
Fri 09 Jan 2015 07:27:50 AM UTC, comment #1: 

in commit b1e2d0b the method is installed in surface_menu.centerline by calling and running install_trisurface_methods from pyformex.plugins.vmtk_itf

francesco <francio>
Project Member
Mon 22 Dec 2014 03:21:50 PM UTC, original submission:  

The function centerline from the vmtk_itf is not automatically loaded when calling the method TriSurface.centerline, so it gives an error. This can be confusing especially using the GUI option.

the plugin vmtk_itf need to be imported first to install centerline as a TriSurface method.

Is it meant to be like that, or is it a bug?

francesco <francio>
Project Member

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by bverheg (Posted a comment)
  • -email is unavailable- added by francio (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-01-09 bverheg StatusFixed => In Progress
        Open/ClosedClosed => Open
    2015-01-09 francio StatusNone => Fixed
        Open/ClosedOpen => Closed

    Back to the top


    Powered by Savane 3.5