DeepSpeed icon indicating copy to clipboard operation
DeepSpeed copied to clipboard

Published 20 hours ago • verified publisher icon microsoft

inference: remove unused _validate_args function

Open nelyahu opened this issue 9 months ago • 4 comments

nelyahu avatar May 08 '24 08:05 nelyahu

@nelyahu, thanks for the cleanup work.

tjruwase avatar May 13 '24 10:05 tjruwase

@loadams this PR was removed from merge queue for some reason. can you check?

nelyahu avatar May 27 '24 12:05 nelyahu

@loadams this PR was removed from merge queue for some reason. can you check?

Hi @nelyahu - yes, I removed it because I wanted to make the pydantic changes first before merging, is that okay?

loadams avatar May 28 '24 16:05 loadams

@loadams this PR was removed from merge queue for some reason. can you check?

Hi @nelyahu - yes, I removed it because I wanted to make the pydantic changes first before merging, is that okay?

@loadams Sure! thanks :)

nelyahu avatar May 28 '24 19:05 nelyahu