gehring / fax

MIT License
78 stars 9 forks source link

Change argument order to facilitate unpacking #3

Closed pierrelux closed 5 years ago

pierrelux commented 5 years ago

This change breaks the API for cga_lagrange_min. The purpose of this change is to make argument unpacking easier by grouping the learning rate parameters together. The required parameter lagrangian is now the first argument and lr_func is the second. cga_test.py has been updated accordingly and the tests pass.