Skip to content

Commit

Permalink
[RLlib] Add missing sampler_results key to fetch min desired reward…
Browse files Browse the repository at this point in the history
… in RLlib release tests (ray-project#35354)

Signed-off-by: Artur Niederfahrenhorst <[email protected]>
  • Loading branch information
ArturNiederfahrenhorst authored and architkulkarni committed May 16, 2023
1 parent 335c145 commit 21a265c
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions rllib/utils/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -786,9 +786,9 @@ def should_check_eval(experiment):

check_eval = should_check_eval(e)
episode_reward_key = (
"episode_reward_mean"
"sampler_results/episode_reward_mean"
if not check_eval
else "evaluation/episode_reward_mean"
else "evaluation/sampler_results/episode_reward_mean"
)

# For smoke-tests, we just run for n min.
Expand Down Expand Up @@ -904,14 +904,18 @@ def should_check_eval(experiment):
if check_eval:
episode_reward_mean = np.mean(
[
t.metric_analysis["evaluation/episode_reward_mean"]["max"]
t.metric_analysis[
"evaluation/sampler_results/episode_reward_mean"
]["max"]
for t in trials_for_experiment
]
)
else:
episode_reward_mean = np.mean(
[
t.metric_analysis["episode_reward_mean"]["max"]
t.metric_analysis["sampler_results/episode_reward_mean"][
"max"
]
for t in trials_for_experiment
]
)
Expand Down

0 comments on commit 21a265c

Please sign in to comment.