1. 27 Jul, 2022 1 commit
  2. 26 Jul, 2022 2 commits
  3. 25 Jul, 2022 1 commit
  4. 22 Jul, 2022 1 commit
  5. 19 Jul, 2022 2 commits
  6. 14 Jul, 2022 4 commits
  7. 13 Jul, 2022 2 commits
  8. 09 Jul, 2022 1 commit
  9. 08 Jul, 2022 3 commits
  10. 06 Jul, 2022 2 commits
  11. 02 Jul, 2022 1 commit
  12. 30 Jun, 2022 2 commits
  13. 29 Jun, 2022 2 commits
  14. 28 Jun, 2022 2 commits
  15. 27 Jun, 2022 1 commit
  16. 24 Jun, 2022 3 commits
    • Yanghan Wang's avatar
      better organize builtin runner's default config · 07eb7a1e
      Yanghan Wang authored
      Summary:
      Pull Request resolved: https://github.com/facebookresearch/d2go/pull/313
      
      It's not natural to put runner's default config functions under `d2go/utils/`, move them to `d2go/runner/config_defaults.py` and clean things up. This also reduce the inter-sub-package dependencies.
      
      Reviewed By: mattcyu1
      
      Differential Revision: D37407078
      
      fbshipit-source-id: 432644bee4f12306a14bac3dba76ced08b3683aa
      07eb7a1e
    • Mik Vyatskov's avatar
      Only save results to file from rank 0 · f0297b81
      Mik Vyatskov authored
      Summary:
      Pull Request resolved: https://github.com/facebookresearch/d2go/pull/309
      
      Right now multiple machines can try to write to the same output file,
      since they get the same argument. Additionally, on the same machine, several
      outputs can be saved which requires unncessary unpacking. This change makes
      train_net only write output of the rank 0 trainer.
      
      Reviewed By: wat3rBro
      
      Differential Revision: D37310084
      
      fbshipit-source-id: 9d5352a274e8fb1d2043393b12896d402333c17b
      f0297b81
    • Yanghan Wang's avatar
      use runner class instead of instance outside of main · 8051775c
      Yanghan Wang authored
      Summary:
      Pull Request resolved: https://github.com/facebookresearch/d2go/pull/312
      
      As discussed, we decided to not use runner instance outside of `main`, previous diffs already solved the prerequisites, this diff mainly does the renaming.
      - Use runner name (str) in the fblearner, ML pipeline.
      - Use runner name (str) in FBL operator, MAST and binary operator.
      - Use runner class as the interface of main, it can be either the name of class (str) or actual class. The main usage should be using `str`, so that the importing of class happens inside `main`. But it's also a common use case to import runner class and call `main` for things like ad-hoc scripts or tests, supporting actual class makes it easier modify code for those cases (eg. some local test class doesn't have a name, so it's not feasible to use runner name).
      
      Reviewed By: newstzpz
      
      Differential Revision: D37060338
      
      fbshipit-source-id: 879852d41902b87d6db6cb9d7b3e8dc55dc4b976
      8051775c
  17. 22 Jun, 2022 2 commits
  18. 21 Jun, 2022 2 commits
  19. 20 Jun, 2022 1 commit
  20. 18 Jun, 2022 2 commits
  21. 17 Jun, 2022 3 commits