Sagemaker environment vars should not be required
Open
- Dominant language
- Python
- Stars
- 625
- Forks
- 103
- PR merge metrics
- No merged PRs in 30d
Description
USE_SAGEMAKER is checked at the beginning of train.py but not in the arg code which includes a few sagemaker variables. Easy enough to comment out but would be nice just to pull the USE_SAGEMAKER through completely in the code for all variables.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.