Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

few.model() and few.print_model() #4

Closed
TheodoreGalanos opened this issue Jan 28, 2017 · 4 comments
Closed

few.model() and few.print_model() #4

TheodoreGalanos opened this issue Jan 28, 2017 · 4 comments

Comments

@TheodoreGalanos
Copy link

Hello!

Thanks for sharing your work, this is really cool!

I was wondering if you could provide a bit of explanation as to the difference between these two outputs of the algorithm.

Also, is there any (outside) documentation on all this?

Thanks in advance!

Kind regards,
Theodore.

@lacava
Copy link
Owner

lacava commented Jan 29, 2017

thanks for your interest!
currently, the difference is that print_model() will return a model complete with coefficients generated from the paired ML, e.g.
-2.5 * sin(x_1x2) +
-4.56 * log(x_1)

whereas model() will just return a list of engineered features. however, these should probably be merged in the future or clarified - maybe renaming model() to features() or just storing a list of features as strings at the end of fit().

@lacava
Copy link
Owner

lacava commented Jan 29, 2017

regarding the documentation, there should be some very soon!

@TheodoreGalanos
Copy link
Author

TheodoreGalanos commented Jan 29, 2017

@lacava Thanks a lot for the clarification, it makes sense.

I'll keep testing and come back with hopefully useful comments, if any.

Kind regards,
Theodore.

@lacava
Copy link
Owner

lacava commented Jan 31, 2017

a7dfb92 adds some outside documentation.

@lacava lacava closed this as completed Jan 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants