Closed MatteoRobbiati closed 4 months ago
@MatteoRobbiati I think this is integrated into #52 and #55 ? I closed #34 because it will be impossible to merge that. Please close or move the target branch to those of Andrea, thanks!
@MatteoRobbiati I think this is integrated into #52 and #55 ? I closed #34 because it will be impossible to merge that. Please close or move the target branch to those of Andrea, thanks!
Not yet but we are going to integrate it soon, thus this one can be closed. @andrea-pasquale can you confirm my modifications done in utils.py
have been migrated into your branches?
Yes, to be sure you can rebase this PR to main and I will merge everything here.
You can try to run stuff in #55 using --optimization_method
to double check.
This leave the possibility to use the old GD but also enables to use more optimizers.