-
Notifications
You must be signed in to change notification settings - Fork 94
Added LBFGS optimizer for Fusion #81
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
Conversation
Hi @e-eight, once again, thanks for the PR :-) It looks like the CI failed, could you check what's wrong with your code according to the details? Besides, you can run pytest locally with the command |
Thanks for your prompt fix, I will validate the performance of |
Hi @e-eight, the result looks good, the performance of |
Hi! I have used the LBFGS optimizer for small datasets, but always with a single model, never with an ensemble (which is why I wanted to use this package in the first place 🙂). Here is a simple example of polynomial regression where the LBFGS optimizer gives much faster convergence than Adam or SGD. Model:
Data:
Optimizer:
Hope this helps! |
According to your experience, does the
|
Hi @e-eight, wondering that if there is any problem since you have not replied in 5 days ;-) Feel free to tell me if you need any help. |
Sorry @xuyxu. Been a little busy these last few days. Will try the fusion regression and get back to you soon. :) |
Never mind, simply contribute to this PR on your own pace. |
Hi @xuyxu. Finally got some time to try it out. The |
Nice try @e-eight ! Sure, could you further check if there is any problem when modifying |
@xuyxu I tried it for |
resolves #79