-
-
Notifications
You must be signed in to change notification settings - Fork 309
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
Math addition transformer #91
Math addition transformer #91
Conversation
merge with solegalli source repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @michalgromiec thank you again for the great work. Please see my comments. I also think we should change the name of the python script, to maybe mathematical_combination.py.
I think mathematical_transformers.py is too similar to the script variable_transformer.py, which applies math transformations variables per variables, and users may get confused.
Cheers
Sole
FIX change transformer name from AdditionTransfomer to MathematicalVariableCombinator FEAT variables_set_alias attribute added
TESTS: fixing to reflect code changes
Hi @michalgromiec thanks a lot for the amount of work. Could I also ask the following: Because this is a new feature that adds functionality at the moment not present in Feature-engine we should also add a bit of information to the documentation. Could I also ask you to include the following files:
Test that the docs run and build successfully. Thanks a lot! |
…maticalVariable REFACTOR: attribute variables_set_alias renamed to new_variables_names FEAT: added build of dictionary of new feature names and their operations FEAT: added check of equal length of operations and new_feature_names DOCS: updated docstrings for mathematical combinator TESTS: added tests to cover above changes
fixed rst underline, changed tests name, removed dup test
@michalgromiec all done! Thank you! |
issue #69 - implementation of AdditionTransformer from mathematical transformation