DeepSpeed
Handle PermissionError in os.chmod Call - Update engine.py
#4139

Merged

Handle PermissionError in os.chmod Call - Update engine.py #4139

loadams merged 10 commits into deepspeedai:master from M-Chris:fix-chmod-permission-issue

Update engine.py

48be0066

M-Chris requested a review from

jeffra 2 years ago

M-Chris requested a review from

tjruwase 2 years ago

M-Chris changed the title ~~Update engine.py~~ Handle PermissionError in os.chmod Call - Update engine.py 2 years ago

Change to use logger

9cdf107c

Split permissions out and add unit test

4ba40470

M-Chris requested a review from

mrwyattii 2 years ago

UnitTest(use DistTestClass) + trailing whitespace

9c115fe5

update unit test

c242778f

tjruwase commented on 2023-08-14

UT parametrize 1, 2 ,3

220ebef1

tjruwase approved these changes on 2023-08-14

loadams commented on 2023-08-14

trim white space from unit test

f199f3c1

change to PermissionError

492d9aea

loadams approved these changes on 2023-08-14

run pre-commit formats

3b43a94f

loadams enabled auto-merge 2 years ago

disabled auto-merge 2 years ago
Manually disabled by user

loadams commented on 2023-08-14

Catch FileNotFoundError & PermissionError

4a6716e5

loadams enabled auto-merge 2 years ago

loadams merged 629b2039 into master 2 years ago

Reviewers

loadams

tjruwase

jeffra

mrwyattii

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

DeepSpeed Handle PermissionError in os.chmod Call - Update engine.py #4139 Merged

Handle PermissionError in os.chmod Call - Update engine.py #4139

DeepSpeed
Handle PermissionError in os.chmod Call - Update engine.py
#4139

Merged