Skip to content

Commit fa04308

Browse files
authored
Remove redundant log for run cli (microsoft#2194)
## Describe your changes Logs from WorkflowRunCommand is duplicated from Engine log. ## Checklist before requesting a review - [ ] Add unit tests for this change. - [ ] Make sure all tests can pass. - [ ] Update documents if necessary. - [ ] Lint and apply fixes to your code by running `lintrunner -a` - [ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes. ## (Optional) Issue link
1 parent 4021ed1 commit fa04308

2 files changed

Lines changed: 1 addition & 5 deletions

File tree

olive/cli/run.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,5 @@ def run(self):
6565

6666
if self.args.list_required_packages is True:
6767
print("Required packages listed!")
68-
elif workflow_output.has_output_model():
69-
print(f"Model is saved at {self.args.output_path}")
70-
else:
71-
print("No output model produced. Please check the log for details.")
7268

7369
return workflow_output

olive/engine/engine.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ def run(
256256
if len(accelerator_output_dir_list) > 1 and self.skip_saving_artifacts:
257257
[shutil.rmtree(folder) for folder in accelerator_output_dir_list if folder.exists()]
258258
else:
259-
logger.warning("No output model")
259+
logger.warning("No output model produced. Please check the log for details.")
260260

261261
return workflow_output
262262

0 commit comments

Comments
 (0)