-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Replace mlflow.pyfunc.load_pyfunc
with mlflow.pyfunc.load_model
#5290
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
Signed-off-by: harupy <hkawamura0130@gmail.com>
133eb29
to
19561d8
Compare
Signed-off-by: harupy <hkawamura0130@gmail.com>
harupy
commented
Jan 21, 2022
mlflow/pyfunc/__init__.py
Outdated
@@ -668,31 +667,6 @@ def load_model(model_uri: str, suppress_warnings: bool = True, dst_path: str = N | |||
return PyFuncModel(model_meta=model_meta, model_impl=model_impl) | |||
|
|||
|
|||
@deprecated("mlflow.pyfunc.load_model", 1.0) |
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.
FYI: MLflow 1.0 was released on Jun 4, 2019:
https://pypi.org/project/mlflow/1.0.0
WeichenXu123
approved these changes
Jan 21, 2022
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.
LGTM
Signed-off-by: harupy <17039389+harupy@users.noreply.github.com>
Signed-off-by: harupy <17039389+harupy@users.noreply.github.com>
mlflow.pyfunc.load_pyfunc
mlflow.pyfunc.load_pyfunc
with mlflow.pyfunc.load_model
erensahin
pushed a commit
to erensahin/mlflow
that referenced
this pull request
Apr 11, 2022
…lflow#5290) * Remove deprecated mlflow.pyfunc.load_pyfunc Signed-off-by: harupy <hkawamura0130@gmail.com> * fix doc Signed-off-by: harupy <hkawamura0130@gmail.com> * lint Signed-off-by: harupy <hkawamura0130@gmail.com> * import load_model Signed-off-by: harupy <hkawamura0130@gmail.com> * revert Signed-off-by: harupy <17039389+harupy@users.noreply.github.com> * replace load_pyfunc in test_transformer_model_export Signed-off-by: harupy <17039389+harupy@users.noreply.github.com>
Merged
5 tasks
ariefrahmansyah
added a commit
to caraml-dev/merlin
that referenced
this pull request
Mar 3, 2023
…se image (#356) <!-- Thanks for sending a pull request! Here are some tips for you: 1. Run unit tests and ensure that they are passing 2. If your change introduces any API changes, make sure to update the e2e tests 3. Make sure documentation is updated for your PR! --> **What this PR does / why we need it**: <!-- Explain here the context and why you're making the change. What is the problem you're trying to solve. ---> The current implementation of the new base image for batch predictor upgrade the MLflow version to 1.30.0, however, there's a backward incompatibility in the loading model of SparkModel: mlflow/mlflow#5290 The SparkModelCache.get_or_load in mlflow>=1.25.0 returns a tuple instead of a pyfunc object. To make it more predictable, we also pin mlflow version in pyfunc's base image to 1.23.0 to match merlin-sdk's requirements. **Does this PR introduce a user-facing change?**: <!-- If no, just write "NONE" in the release-note block below. If yes, a release note is required. Enter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string "action required". For more information about release notes, see kubernetes' guide here: http://git.k8s.io/community/contributors/guide/release-notes.md --> ```release-note NONE ``` **Checklist** - [ ] Added unit test, integration, and/or e2e tests - [ ] Tested locally - [ ] Updated documentation - [ ] Update Swagger spec if the PR introduce API changes - [ ] Regenerated Golang and Python client if the PR introduce API changes
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.
Signed-off-by: harupy hkawamura0130@gmail.com
What changes are proposed in this pull request?
Replace
mlflow.load_pyfunc
(deprecated) withmlflow.pyfunc.load_model
.How is this patch tested?
Existing tests
Does this PR change the documentation?
ci/circleci: build_doc
check. If it's successful, proceed to thenext step, otherwise fix it.
Details
on the right to open the job page of CircleCI.Artifacts
tab.docs/build/html/index.html
.Release Notes
Is this a user-facing change?
mlflow.pyfunc.load_pyfunc
has been removed.What component(s), interfaces, languages, and integrations does this PR affect?
Components
area/artifacts
: Artifact stores and artifact loggingarea/build
: Build and test infrastructure for MLflowarea/docs
: MLflow documentation pagesarea/examples
: Example codearea/model-registry
: Model Registry service, APIs, and the fluent client calls for Model Registryarea/models
: MLmodel format, model serialization/deserialization, flavorsarea/projects
: MLproject format, project running backendsarea/scoring
: MLflow Model server, model deployment tools, Spark UDFsarea/server-infra
: MLflow Tracking server backendarea/tracking
: Tracking Service, tracking client APIs, autologgingInterface
area/uiux
: Front-end, user experience, plotting, JavaScript, JavaScript dev serverarea/docker
: Docker use across MLflow's components, such as MLflow Projects and MLflow Modelsarea/sqlalchemy
: Use of SQLAlchemy in the Tracking Service or Model Registryarea/windows
: Windows supportLanguage
language/r
: R APIs and clientslanguage/java
: Java APIs and clientslanguage/new
: Proposals for new client languagesIntegrations
integrations/azure
: Azure and Azure ML integrationsintegrations/sagemaker
: SageMaker integrationsintegrations/databricks
: Databricks integrationsHow should the PR be classified in the release notes? Choose one:
rn/breaking-change
- The PR will be mentioned in the "Breaking Changes" sectionrn/none
- No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" sectionrn/feature
- A new user-facing feature worth mentioning in the release notesrn/bug-fix
- A user-facing bug fix worth mentioning in the release notesrn/documentation
- A user-facing documentation change worth mentioning in the release notes