Skip to content
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

Fix: Removed get_experiment_by_name as not required #5808

Merged
merged 1 commit into from
Apr 30, 2022
Merged

Fix: Removed get_experiment_by_name as not required #5808

merged 1 commit into from
Apr 30, 2022

Conversation

ketangangal
Copy link
Contributor

@ketangangal ketangangal commented Apr 30, 2022

What changes are proposed in this pull request?

Fixes #5808

URL(s) with the issue:
https://mlflow.org/docs/latest/_modules/mlflow/tracking/fluent.html#set_experiment
https://github.com/mlflow/mlflow/blob/master/mlflow/tracking/fluent.py

Change documentation / code example from

import mlflow
# Set an experiment name, which must be unique and case sensitive.
mlflow.set_experiment("Social NLP Experiments")
# Get Experiment Details
experiment = mlflow.get_experiment_by_name("Social NLP Experiments")
print("Experiment_id: {}".format(experiment.experiment_id))
print("Artifact Location: {}".format(experiment.artifact_location))
print("Tags: {}".format(experiment.tags))
print("Lifecycle_stage: {}".format(experiment.lifecycle_stage))

to

import mlflow
# Set an experiment name, which must be unique and case-sensitive.
experiment = mlflow.set_experiment("Social NLP Experiments")
# Get Experiment Details
print("Experiment_id: {}".format(experiment.experiment_id))
print("Artifact Location: {}".format(experiment.artifact_location))
print("Tags: {}".format(experiment.tags))
print("Lifecycle_stage: {}".format(experiment.lifecycle_stage))

@github-actions
Copy link

@ketangangal Thanks for the contribution! The DCO check failed. Please sign off your commits by following the instructions here: https://github.com/mlflow/mlflow/runs/6239934586. See https://github.com/mlflow/mlflow/blob/master/CONTRIBUTING.rst#sign-your-work for more details.

Signed-off-by: ketanGangal <ketangangal98@gmail.com>
@harupy harupy added the rn/none List under Small Changes in Changelogs. label Apr 30, 2022
Copy link
Member

@harupy harupy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@harupy harupy merged commit d0d307f into mlflow:master Apr 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn/none List under Small Changes in Changelogs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants