Error logging #30
Merged
Error logging #30
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I'm not sure if it the right way to do that... but it would be nice to have error logged as well. |
gfournier
pushed a commit
that referenced
this pull request
Feb 28, 2020
gfournier
added a commit
that referenced
this pull request
Feb 28, 2020
* bump version to 0.1.3 * bump version to 0.1.4-dev * massive black reformating (#18) * fix bug when type_of_problem is setted (#21) * fix bug when type_of_problem is setted * add default * add specific test * Fix numericalencoder (#22) * fix NumericalEncoder with default values * fix NumericalEncoder with default values * Fix doc (#26) * ignore .bat file to create doc * fix doc * comment in english * clean test * requirements pandas >= 0.23 (#27) * node -> nodes (was deprecated and is now absent) (#31) * Add test picklable (#23) * test numerical encoder is picklable * test numerical encoder is picklable * test target encoder is picklable * black * black * remove warning printing * add test : unpickled object behave like original object * improve auto ml doc (#30) * re-index 'fit_params' that are indexable * change version 0.1.5 * conversion model to json (#36) * v0.1.6-dev * add conversion model to json : 'param_from_sklearn_model' + corresponding tests * add new numpy type to be cast to python type * test if object can be json serialized * refactoring of columns selection (#29) * add conversion model to json : 'param_from_sklearn_model' + corresponding tests * change wrapper, 'drop_used_columns' and 'drop_unused_columns' * temp : remove useless attribute * temp : fix test * allow selector to select of type of variable among TypeOfVariables.CAT / TEXT / NUM * change default for numerical encoder * change test * temp : new test * renamming * comments * clean docstring * change text models * change 'base' models * change corresponding tests * add numpy array support * fix tests * fix test * fix random_forest_addins columns_to_use * fix Targetencoder * fix special case when no column to pass to the model * typo * fix get_feature_names * allow not to raise when shape between fit and transform differs * corresponding tests * cleanning * fix doc + default * rename * fix registration * black reformat * clean * add helper method * temp add fitting test * clean * add test : try to fit model * add custom default hyper-parameters * fix inf * clean * add test not inf CdfScaler * cap number of component to nb of rows * fix seed by default * clean * make CdfScaler to very small, almost equal values * test very close and very small values * cleanning * divers * add min_count param * more data in test * * remove cast of string that can be parsed * corresponding test * change version 1.0.0 * dev version 1.0.1-dev * change version 0.2.0 * dev 0.2.1-dev
gfournier
approved these changes
Mar 9, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add "try except" block to catch error, log it and re-raise it