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.

Sign up for GitHub

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

[rllib] Fix storage-path related tests #38947

Merged
merged 18 commits into from
Aug 29, 2023
Merged

[rllib] Fix storage-path related tests #38947

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
c12b5ef
test_api, Durable
Aug 25, 2023
8643515
[rllib] Fix storage path tests
Aug 26, 2023
44de41f
Merge branch 'master' into rllib/storage-tests
matthewdeng Aug 26, 2023
15cd1e3
Update tests
Aug 26, 2023
91dcd19
evaluate
Aug 26, 2023
627a4a5
Merge remote-tracking branch 'upstream/master' into rllib/storage-tests
Aug 26, 2023
85b45b3
restore from checkpoint
Aug 26, 2023
6995aad
eval
Aug 27, 2023
d5b2d58
custom experiment
Aug 27, 2023
23ba3ff
eval
Aug 27, 2023
dd2a315
eval from checkpoint path
Aug 27, 2023
7dd026e
Merge remote-tracking branch 'upstream/master' into rllib/storage-tests
Aug 28, 2023
ee6e8f9
Fix ray client test
Aug 28, 2023
8276bcd
restore from path
Aug 28, 2023
0d4353c
checkpoint dir
Aug 28, 2023
c6be114
Merge remote-tracking branch 'upstream/master' into rllib/storage-tests
Aug 29, 2023
a7e2ea2
fix merge conflict
Aug 29, 2023
9a49f3d
doc updates
Aug 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
eval 
Signed-off-by: Kai Fricke <kai@anyscale.com>
  • Loading branch information
Kai Fricke committed Aug 27, 2023
commit 6995aadd2ffdd8bcbe26a50230e9b64a8a5faa0b
2 changes: 1 addition & 1 deletion rllib/evaluate.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ def run(
# Load state from checkpoint, if provided.
if checkpoint:
restore_result = _TrainingResult(
checkpoint=Checkpoint.from_directory(str(checkpoint)), metrics={}
checkpoint=Checkpoint.from_directory(config_dir), metrics={}
)
algorithm.restore(restore_result)

Expand Down
1 change: 1 addition & 0 deletions rllib/examples/documentation/saving_and_loading_algos_and_policies.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@
)

my_ma_algo = my_ma_config.build()
my_ma_algo.train()

ma_checkpoint_dir = my_ma_algo.save().checkpoint.path

Expand Down
2 changes: 1 addition & 1 deletion rllib/tests/test_rllib_train_and_evaluate.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ def evaluate_test(algo, env="CartPole-v1", test_episode_rollout=False):
os.popen(
'python {}/evaluate.py --run={} "{}" --steps=10 '
'--out="{}/rollouts_10steps.pkl"'.format(
rllib_dir, algo, str(Path(checkpoint_path).parent), tmp_dir
rllib_dir, algo, checkpoint_path, tmp_dir
)
).read()
if not os.path.exists(tmp_dir + "/rollouts_10steps.pkl"):
Expand Down