Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • M metaseq
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 95
    • Issues 95
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 41
    • Merge requests 41
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Administrator
  • metaseq
  • Merge requests
  • !543

Fix test checkpoint saving

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge fix_test_checkpoint_saving into main Dec 03, 2022
  • Overview 5
  • Commits 11
  • Pipelines 0
  • Changes 2

Created by: zycalice

Patch Description related to issue https://github.com/facebookresearch/metaseq/issues/516

  1. fixed distributed_main_mock by patching save_checkpoint in trainer
  2. replaced partial with partial method for patching on class method (the log_to_events mock now print out message instead of info due to the presence of self
  3. note that now loss matching is changed to matching to the 1st digit

Testing steps Ran python metaseq/gpu_tests/test_checkpoint_saving.py output: . Ran 1 test in 52.583s

OK

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_test_checkpoint_saving