Update dependencies check: package whitelist #417
Merged
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
dsherry
commented
Feb 25, 2020
| @@ -177,7 +177,8 @@ jobs: | |||
| command: | | |||
| source test_python/bin/activate | |||
| mkdir /tmp/dependencies_updated_artifacts | |||
| pip freeze | grep -v "FeatureLabs/evalml.git" > /tmp/dependencies_updated_artifacts/current_dependencies.txt | |||
| export whitelist="pandas|numpy|scikit|xgboost|catboost|boto|category-encoders|cloudpickle|dask|distributed|pyzmq|statsmodels" | |||
There was a problem hiding this comment.
There's gotta be a better way to do this but this works 🤷♂️
There was a problem hiding this comment.
What else should be in this list? I'd include anything which could cause mathematical or numerical precision errors, or generally anything which could cause a unit test failure.
Codecov Report
@@ Coverage Diff @@
## master #417 +/- ##
=======================================
Coverage 97.39% 97.39%
=======================================
Files 103 103
Lines 3224 3224
=======================================
Hits 3140 3140
Misses 84 84Continue to review full report at Codecov.
|
Closed
jeremyliweishih
approved these changes
Feb 25, 2020
angela97lin
pushed a commit
that referenced
this pull request
Feb 25, 2020
* Add a whitelist for update_deps check * Remove from expected * Update deps * Changelog
Merged
angela97lin
added a commit
that referenced
this pull request
Apr 3, 2020
* creating new binary / multiclass variants of pipelines, duplicating code for now * moving common fxns back to pipeline base * more moving around fxns in pipelines classes * capping xgboost * fixing typo * more cleanup, making predict_proba standard regardless of binary/multiclass * renaming other_objectives to objectives * updating score's objective parameter to calculate all objectives, not just additional * removing self.objective for scoring * removing objectives from pipeline initialization, adding objective as predict param * remove xgboost cap from branch * changelog * capping xgboost on local branch since tests timing out * cleaning up * more cleanup * reverting requirements file * adding classification pipeline subclass, cleaning up via PR comments * more cleanup for docstrings * more cleanup of changelog and comments * putting tests in subfolders and adding few more tests * Update dependencies (#412) * Update latest dependencies * Hide features with zero importance in plot by default (#413) * adding functionality and test * changelog and adding boolean param * Update dependencies check: package whitelist (#417) * Add a whitelist for update_deps check * Remove from expected * Update deps * Changelog * adding skeleton for subclasses * fixing test and linting * updating change from master * fixing fixture * cherry picked wip remove ROC and confusion matrix * fixing merge * fixing merge * cleaning up * make test use static attribute instead of instance' * deleting needs_fitting * updating code to use new objective classes, still broken * updating threshold, still need to clean up tests * comment out for now * more cleanup * cleaning up * more cleanup * still more cleanup * fixing plot unsuccessful merging * more cleanup but still some things to work out * cleaning up using multiclass objectives for binary classification problems * fixing typo with recall and cleanup * cleaning up * adding default * some more cleaning up * removing irrelevant test * forgot to add attribute, breaking things again * cleanup and change objective of test * removing objective from predict * more cleanup :d * remove unused attribute * cleaning up via comments * more comments * changelog * order of decorators changed * fixing copy and paste err * update random state for binary class pipelines * updating objective * typo * fixing? * fixing imports * fixing tests * adding objective as parameter for predict, removing for fit * cleaning up test * more fixing test * minor linting, need more to go * more cleanup * forgot to fix test * more merging :x * starting to add tuning logic to automl * changelog * cleaning up * change conditional for objective split * cleaning up docstrings * forgot to use classificationobjective class... * add additional cond * adding tests * cleanup * removing decision function for multiclass * updating via comments * removing classification_objective file * add test + more updates * use cls instead for pep8 standards * updating can_optimize to property * update score * fix tests * minor cleanup from comments * updating predict behavior * add separate objective check * fixing some merge conflicts cont * add fraud test * patching * remove old test * updating for now * add another test * add more tests * adding test structure, still need to fix * adding test * fix iloc * fix tests * fix import * fix test? * removing can_optimize_threshold * linting * update docs a little * remove accuracy * add more doc fixes * move binary and multi pipelines in api ref * revert components notebook * updating from comments * oops, fix none set * update docstring * update api ref? * addressing comments * revert and update * update docstring * updating docstrings and lint * updating unnecessary call to constructor * pushing empty commit to refresh Co-authored-by: Jeremy Shih <jeremyliweishih@gmail.com> Co-authored-by: Dylan Sherry <sharshofski@gmail.com>
angela97lin
added a commit
that referenced
this pull request
Apr 13, 2020
* Objectives API: Create new binary / multiclass pipeline classes and remove objectives from pipeline classes (#405) * Objectives API: Remove ROC and confusion matrix as objectives (#422) * Change `score` output to return one dictionary (#429) * Create binary and multiclass objective classes (#504) * Update dependencies (#412) * Hide features with zero importance in plot by default (#413) * Update dependencies check: package whitelist (#417) * Add fixes necessary for docs to build for improved objectives project (#605) * Remove calculating plot metrics from AutoML (#615)
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.
Only do the check for a specific whitelist of packages. To reduce noise.