Skip to content

Commit

Permalink
Fix duplicate progress output. (ray-project#6497)
Browse files Browse the repository at this point in the history
  • Loading branch information
ujvl authored and ericl committed Dec 16, 2019
1 parent 5719a05 commit e38b25e
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
6 changes: 4 additions & 2 deletions python/ray/tune/progress_reporter.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ def report(self, trial_runner):
messages = [
"== Status ==",
memory_debug_str(),
trial_runner.debug_string(delim=delim),
trial_runner.scheduler_alg.debug_string(),
trial_runner.trial_executor.debug_string(),
trial_progress_str(trial_runner.get_trials(), fmt="html"),
trial_errors_str(trial_runner.get_trials(), fmt="html"),
]
Expand All @@ -68,7 +69,8 @@ def report(self, trial_runner):
messages = [
"== Status ==",
memory_debug_str(),
trial_runner.debug_string(),
trial_runner.scheduler_alg.debug_string(),
trial_runner.trial_executor.debug_string(),
trial_progress_str(trial_runner.get_trials()),
trial_errors_str(trial_runner.get_trials()),
]
Expand Down
4 changes: 4 additions & 0 deletions python/ray/tune/trial_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,10 @@ def __init__(self,
self._local_checkpoint_dir,
TrialRunner.CKPT_FILE_TMPL.format(self._session_str))

@property
def scheduler_alg(self):
return self._scheduler_alg

def _validate_resume(self, resume_type):
"""Checks whether to resume experiment.
Expand Down

0 comments on commit e38b25e

Please sign in to comment.