Break export() down into 3 separate functions about tpot

[pronojitsaha]

@rhiever While I am working on this, can you provide some inputs on the 'type' and 'contents' of the following just to be sure
-exported_pipeline,
-pipeline_list, and
-operator_text

[rhiever]

@pronojitsaha, it may be best to hold off on this issue until we see what affect @dmarx's refactor (see #43 (comment)) has on the export function. I suspect it will shrink the export function significantly.

[pronojitsaha]

Ok. I see that @dmarx's implementation still has the first two functions in export i.e.

• Replace all of the mathematical operators with their results
• Unroll the nested function calls into serial code

Should I break that into separate functions, it will streamline export() further and then incorporate @dmarx's refactor on the third part i.e. 'Replace the function calls with their corresponding Python code'?

[dmarx]

@pronojitsaha: Sounds good to me

[pronojitsaha]

@dmarx Ok..great! Will get it done tomorrow.

[rhiever]

👍

[pronojitsaha]

@rhiever While we await the integration of #63, it will be good to break the final part i.e. 'Replace the function calls with their corresponding Python code' into a separate function or a code that the main tpot file imports from for improved code readability and comprehensibility significantly (similar way that I did the first two). I already have a basic structure, if you want can create a PR for it and you can look at it?

[rhiever]

Sounds good to me.

On Wednesday, December 23, 2015, PRONOjit Saha [email protected]
wrote:

@rhiever https://github.com/rhiever While we await the integration of
#63 #63, it will be good to break
the final part i.e. 'Replace the function calls with their corresponding
Python code' into a separate function or a code that the main tpot file
imports from for improved code readability and comprehensibility
significantly (similar way that I did the first two). I already have a
basic structure, if you want can create a PR for it and you can look at it?

—
Reply to this email directly or view it on GitHub
#44 (comment).

Randal S. Olson, Ph.D.
Postdoctoral Researcher, Institute for Biomedical Informatics
University of Pennsylvania

E-mail: [email protected] | Twitter: @randal_olson
https://twitter.com/randal_olson
http://www.randalolson.com