ml-agents
ml-agents copied to clipboard
Remove unnecessary learning_rate arguments
Proposed change(s)
Removing the learning rate argument from some of our optimizers. These optimizers update their learning rate before each update using the scheduler. Passing the learning rate as argument is confusing because this argument does not actually do anything (I got tricked once).
Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)
Types of change(s)
- [ ] Bug fix
- [ ] New feature
- [x] Code refactor
- [ ] Breaking change
- [ ] Documentation update
- [ ] Other (please describe)
Checklist
- [ ] Added tests that prove my fix is effective or that my feature works
- [ ] Updated the changelog (if applicable)
- [ ] Updated the documentation (if applicable)
- [ ] Updated the migration guide (if applicable)