fairseq2 icon indicating copy to clipboard operation
fairseq2 copied to clipboard
verified publisher icon facebookresearch

Add RayClusterHandler to enable fairseq2 run on Ray

Open dongwang218 opened this issue 1 year ago • 1 comments

What does this PR do? Please describe: Allow fairseq2 to run inside ray cluster.

Does your PR introduce any breaking changes? If yes, please list them: List of all backwards-incompatible changes.

Check list:

  • [N ] Was the content of this PR discussed and approved via a GitHub issue? (no need for typos or documentation improvements)
  • [ Y] Did you read the contributor guideline?
  • [ Y] Did you make sure that your PR does only one thing instead of bundling different changes together?
  • [ Y] Did you make sure to update the documentation with your changes? (if necessary)
  • [ Y] Did you write any new necessary tests?
  • [ Y] Did you verify new and existing tests pass locally with your changes?
  • [ N] Did you update the CHANGELOG? (no need for typos, documentation, or minor internal changes)

dongwang218 avatar Mar 28 '25 02:03 dongwang218

nice contribution ! can you add a simple integration test and some doc so everyone could start using this integration with Ray ?

artemru avatar Mar 28 '25 15:03 artemru