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

Quentin leboutet/point goal nav training #314

Merged

Conversation

quentin-leboutet
Copy link
Collaborator

Integration of the point goal navigation training pipeline

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@thias15
Copy link
Collaborator

thias15 commented Dec 7, 2022

Please fix style and test if Autopilot still works the same.

@quentin-leboutet quentin-leboutet marked this pull request as ready for review December 9, 2022 15:08
@thias15 thias15 self-assigned this Dec 12, 2022
thias15 and others added 8 commits December 17, 2022 10:49
- save last, best train, best val checkpoints instead of keeping so many
- keep track of initial epoch when resuming to fix logging
- unify batch visualization
- needs to overwrite policy type since it's not integrated with GUI and returns `None`
@thias15 thias15 merged commit ca8e129 into isl-org:master Dec 19, 2022
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.

None yet

2 participants