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.

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

Extract ckpt_id from ckpt_path #493

Merged
merged 3 commits into from
Oct 8, 2020
Merged

Extract ckpt_id from ckpt_path #493

merged 3 commits into from
Oct 8, 2020

Conversation

Skylion007
Copy link
Contributor

Motivation and Context

It means the BaseTrainer will try to extract ckpt_idx if you are evaluating on a single checkpoint. This means you can evaluate on multiple single checkpoints in a folder and have them output videos to the same folder without overwriting each other.

How Has This Been Tested

Evaluated ckpts locally, tests pass.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have completed my CLA (see CONTRIBUTING)
  • All new and existing tests passed.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 5, 2020
@Skylion007 Skylion007 merged commit 7766bf5 into master Oct 8, 2020
@Skylion007 Skylion007 deleted the eval_get_ckpt_idx branch October 8, 2020 17:23
srama2512 pushed a commit that referenced this pull request Mar 15, 2023
* OBB fixes and extends. Also added error val to sample navigable point
* Make getRandomNavigablePoint failure value constant.

Co-Authored-By: Erik Wijmans <etw@gatech.edu>

Co-authored-by: Erik Wijmans <ewijmans2@gmail.com>
dannymcy pushed a commit to dannymcy/habitat-lab that referenced this pull request Jul 8, 2024
* Extract ckpt_id from ckpt_path

* rename util function

* Move defrost
HHYHRHY pushed a commit to SgtVincent/EMOS that referenced this pull request Aug 31, 2024
* Extract ckpt_id from ckpt_path

* rename util function

* Move defrost
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants