Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

SSL Online evaluator: save checkpoints without DDP wrapper #623

Merged
merged 12 commits into from
Jan 6, 2022

Conversation

ant0nsc
Copy link
Contributor

@ant0nsc ant0nsc commented Dec 15, 2021

When using DDP, write the wrapped module directly

Please follow the guidelines for PRs contained here. Checklist:

  • Ensure that your PR is small, and implements one change.
  • Add unit tests for all functions that you introduced or modified.
  • Run PyCharm's code cleanup tools on your Python files.
  • Link the correct GitHub issue for tracking.
  • Update the Changelog file: Describe your change in terms of
    Added/Changed/Removed/... in the "Upcoming" section.
  • When merging your PR, replace the default merge message with a description of your PR,
    and if needed a motivation why that change was required.

ozan-oktay
ozan-oktay previously approved these changes Dec 15, 2021
@ant0nsc ant0nsc enabled auto-merge (squash) January 6, 2022 14:12
@ant0nsc ant0nsc merged commit ac6a312 into main Jan 6, 2022
@ant0nsc ant0nsc deleted the antonsc/sslrecovery branch January 6, 2022 14:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants