-
Notifications
You must be signed in to change notification settings - Fork 10
Paper draft #205
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: paper
Are you sure you want to change the base?
Paper draft #205
Conversation
e96129f
to
5b9e859
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## paper #205 +/- ##
========================================
Coverage ? 59.36%
========================================
Files ? 15
Lines ? 1885
Branches ? 0
========================================
Hits ? 1119
Misses ? 766
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This branch must be rebased. |
Co-authored-by: Maxence Gollier <134112149+MaxenceGollier@users.noreply.github.com>
fca3a30
to
2bb4f05
Compare
@dpo you may now begin the review of the paper. |
@MohamedLaghdafHABIBOULLAH You could also remove |
0df3739
to
269bad8
Compare
6373690
to
bc756ee
Compare
@Loptima here is the joss paper. |
@MohamedLaghdafHABIBOULLAH Could you please add a workflow so the PDF is posted in the pull request? |
In fact, I tried it but it doesn’t work. |
can you rebase @MohamedLaghdafHABIBOULLAH ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a quick review for now, other comments will follow
- name: Create release | ||
if: github.event_name == 'push' | ||
uses: rymndhng/release-on-push-action@master | ||
id: release | ||
with: | ||
bump_version_scheme: patch | ||
tag_prefix: v | ||
release_body: "" | ||
use_github_release_notes: true | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
- name: Upload PDF to release | ||
if: github.event_name == 'push' | ||
uses: svenstaro/upload-release-action@v2 | ||
with: | ||
repo_token: ${{ secrets.GITHUB_TOKEN }} | ||
file: paper/paper.pdf | ||
asset_name: joss-draft.pdf | ||
tag: ${{ steps.release.outputs.tag_name }} | ||
overwrite: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, it looks like it does not work.
This is weird, you might need to remove I am not sure.
# References | ||
# Summary | ||
|
||
[RegularizedOptimization.jl](https://github.com/JuliaSmoothOptimizers/RegularizedOptimization.jl) is a Julia [@bezanson-edelman-karpinski-shah-2017] package that implements a family of regularization and trust-region type algorithms for solving nonsmooth optimization problems of the form: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[RegularizedOptimization.jl](https://github.com/JuliaSmoothOptimizers/RegularizedOptimization.jl) is a Julia [@bezanson-edelman-karpinski-shah-2017] package that implements a family of regularization and trust-region type algorithms for solving nonsmooth optimization problems of the form: | |
[RegularizedOptimization.jl](https://github.com/JuliaSmoothOptimizers/RegularizedOptimization.jl) is a Julia [@bezanson-edelman-karpinski-shah-2017] package that implements a family of quadratic regularization and trust-region type algorithms for solving nonsmooth optimization problems of the form: |
|
||
- **Trust-region methods (TR, TRDH)** [@aravkin-baraldi-orban-2022] and [@leconte-orban-2023], | ||
- **Quadratic regularization methods (R2, R2N)** [@diouane-habiboullah-orban-2024] and [@aravkin-baraldi-orban-2022], | ||
- **Levenbergh-Marquardt methods (LM, LMTR)** [@aravkin-baraldi-orban-2024]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- **Levenbergh-Marquardt methods (LM, LMTR)** [@aravkin-baraldi-orban-2024]. | |
- **Levenberg-Marquardt methods (LM, LMTR)** [@aravkin-baraldi-orban-2024]. |
|
||
These methods rely solely on the gradient and Hessian(-vector) information of the smooth part $f$ and the proximal mapping of the nonsmooth part $h$ in order to compute steps. | ||
Then, the objective function $f + h$ is used only to accept or reject trial points. | ||
Moreover, they can handle cases where Hessian approximations are unbounded[@diouane-habiboullah-orban-2024] and [@leconte-orban-2023-2], making the package particularly suited for large-scale, ill-conditioned, and nonsmooth problems. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moreover, they can handle cases where Hessian approximations are unbounded[@diouane-habiboullah-orban-2024] and [@leconte-orban-2023-2], making the package particularly suited for large-scale, ill-conditioned, and nonsmooth problems. | |
Moreover, they can handle cases where Hessian approximations are unbounded [@diouane-habiboullah-orban-2024] and [@leconte-orban-2023-2], making the package particularly suited for large-scale, ill-conditioned, and nonsmooth problems. |
|
||
## Model-based framework for nonsmooth methods | ||
|
||
There exists a way to solve \eqref{eq:nlp} in Julia using [ProximalAlgorithms.jl](https://github.com/JuliaFirstOrder/ProximalAlgorithms.jl), which implements in-place first-order line search–based methods for composite optimization. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
composite
has not been defined
stats = RegularizedExecutionStats(reg_nlp) | ||
|
||
# Solve the problem | ||
solve!(solver_tr, reg_nlp, stats, x = f.meta.x0, atol = 1e-3, rtol = 1e-4, verbose = 10, ν = 1.0e+2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test does not work locally.
For example
Here is the first version of the JOSS paper @dpo @MaxenceGollier and Youssef.
Please note that the
draft-paper.yml
workflow file should be updated to work with the latest versions and to run on pull requests.Here is the pdf: paper.pdf