Thanks for the patch, looks OK except for my comments. Please address those and checkin. Thanks. http://codereview.appspot.com/88095/diff/1/4 File enthought/mayavi/tools/tools.py (right): http://codereview.appspot.com/88095/diff/1/4#newcode108 Line 108: def get_vtk_src(mayavi_object, filter_ok=True): I don't like "filter_ok", "stop_at_filter" sounds better. http://codereview.appspot.com/88095/diff/1/4#newcode116 Line 116: :filter_ok: optionnal boolean flag: if True, the first object spelling: optional http://codereview.appspot.com/88095/diff/1/4#newcode128 Line 128: This function traverses the Mayavi pipeline. Thus the input please clarify what you mean by "the input object" by referring to the `mayavi_object`. http://codereview.appspot.com/88095/diff/1/4#newcode133 Line 133: raise TypeError, 'Cannot find data source for given object' Perhaps the error message can be 'Cannot find data source for given %s'%(mayavi_object) or something more informative.