Change describe to use mlrun/ml-models instead of mlrun/mlrun image #81
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
* Added Pandas Profiling Report Function (mlrun#78) Co-authored-by: Nick Schenone <[email protected]> * round 2 updates (mlrun#77) * add function * describe presto * add describe spark notebook * no need * add iris csv to describe * add iris dataset * add readme * add readme * add readme and split fix * add model * fix ability to run stand alone * add readme and update nb * add .py * update model server and readme * add read me and iris parquet * update model server test and readme * remove describe spark * update naming * update to wasabi * round 2 improvements * update model tester * round 2 updates * update model server * add sklearn classifier with dask * change get header loction * make classifier genreric * Update README.md * Update README.md * add .py .yaml files again * .DS_Store banished * add .py, .yaml, spark describe * add .py .yaml files * update with pip install and .yaml * add .yaml * add .yaml .py files * add .py .yaml files * add readme and docstring * update readme * change work with dask init * update functions - model server pending * update model server * update model servers Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: idan707 <[email protected]> * updates Co-authored-by: nschenone <[email protected]> Co-authored-by: Nick Schenone <[email protected]> Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: idan707 <[email protected]> Co-authored-by: yaron haviv <[email protected]>
Co-authored-by: Nick Schenone <[email protected]>
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Hedingber
added a commit
that referenced
this pull request
Dec 29, 2020
yaronha
added a commit
that referenced
this pull request
Jan 27, 2021
* Development (#79) * Added Pandas Profiling Report Function (#78) Co-authored-by: Nick Schenone <[email protected]> * round 2 updates (#77) * add function * describe presto * add describe spark notebook * no need * add iris csv to describe * add iris dataset * add readme * add readme * add readme and split fix * add model * fix ability to run stand alone * add readme and update nb * add .py * update model server and readme * add read me and iris parquet * update model server test and readme * remove describe spark * update naming * update to wasabi * round 2 improvements * update model tester * round 2 updates * update model server * add sklearn classifier with dask * change get header loction * make classifier genreric * Update README.md * Update README.md * add .py .yaml files again * .DS_Store banished * add .py, .yaml, spark describe * add .py .yaml files * update with pip install and .yaml * add .yaml * add .yaml .py files * add .py .yaml files * add readme and docstring * update readme * change work with dask init * update functions - model server pending * update model server * update model servers Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: idan707 <[email protected]> * updates Co-authored-by: nschenone <[email protected]> Co-authored-by: Nick Schenone <[email protected]> Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: idan707 <[email protected]> Co-authored-by: yaron haviv <[email protected]> * Added Pandas Profiling Report Function (#78) Co-authored-by: Nick Schenone <[email protected]> * updates * Update mlutils imports to use specific import (#80) * Change describe to use mlrun/ml-models instead of mlrun/mlrun image (#81) * Update function.yaml Co-authored-by: Yaron Haviv <[email protected]> Co-authored-by: Nick Schenone <[email protected]> Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: Idan Benaun <[email protected]> Co-authored-by: idan707 <[email protected]> Co-authored-by: yaron haviv <[email protected]> Co-authored-by: Hedingber <[email protected]>
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.
#77 Changed describe notebook to set the image to mlrun/mlrun but for some reason this change didn't happen in the
function.yaml(what actually matters)In #80 I also touched the describe notebook, and therefore reran it to regenerate the
function.yamlso this time it did changed the image.This function uses
mlutilswhich requiresscikit-plotwhich is not installed inmlrun/mlruntherefore we'll have to change back tomlrun/ml-modelsfor now