Skip to content

write complete report after freezing - #282

Merged
kepricon merged 1 commit into
devfrom
dh/make-report-after-freezing
Mar 18, 2021
Merged

write complete report after freezing#282
kepricon merged 1 commit into
devfrom
dh/make-report-after-freezing

Conversation

@kepricon

Copy link
Copy Markdown
Contributor

webapp rely on ExperimentCompletionReport.txt to get the status of the training.
https://github.com/SkymindIO/pathmind-webapp/blob/dev/pathmind-services/src/main/java/io/skymind/pathmind/services/training/cloud/aws/AWSExecutionProvider.java#L158

PathmindAI/pathmind-webapp#2954

need to write the report after freezing.
freezing normally takes 4~6 mins depends on the model.

@kepricon kepricon self-assigned this Mar 18, 2021
@kepricon
kepricon requested review from maxpumperla and slinlee March 18, 2021 18:31
Comment thread nativerl/python/run.py
freeze_trained_policy(env=env_instance, env_name=env_name, callbacks=callbacks, trials=trials,
algorithm=algorithm, output_dir=f"{output_dir}/{algorithm}/freezing", is_discrete=discrete)

write_completion_report(trials=trials, output_dir=output_dir, algorithm=algorithm)

@slinlee slinlee Mar 18, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this output dir point to the policy after the freezing process?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

output_dir is normally work.
work/PPO/ has training result
work/PPO/freezing/ has freezing result
work/ExperimentCompletionReport.txt has a report from https://github.com/SkymindIO/nativerl/blob/dev/nativerl/python/pathmind/utils.py#L17

@kepricon
kepricon merged commit 7d32304 into dev Mar 18, 2021
@kepricon
kepricon deleted the dh/make-report-after-freezing branch March 18, 2021 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants