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

 
 

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

Submitter:  francesco <francio>
Submitted:  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
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    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.9