Skip to content

Conversation

MohamedLaghdafHABIBOULLAH
Copy link
Contributor

@MohamedLaghdafHABIBOULLAH MohamedLaghdafHABIBOULLAH commented Sep 5, 2025

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

Copy link

codecov bot commented Sep 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (paper@686e4d1). Learn more about missing BASE report.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dpo
Copy link
Member

dpo commented Sep 6, 2025

This branch must be rebased.

@MohamedLaghdafHABIBOULLAH
Copy link
Contributor Author

@dpo you may now begin the review of the paper.

@dpo
Copy link
Member

dpo commented Sep 7, 2025

@MohamedLaghdafHABIBOULLAH You could also remove cirrus.yml from this branch.

@MohamedLaghdafHABIBOULLAH
Copy link
Contributor Author

MohamedLaghdafHABIBOULLAH commented Sep 8, 2025

@Loptima here is the joss paper.

@dpo
Copy link
Member

dpo commented Sep 15, 2025

@MohamedLaghdafHABIBOULLAH Could you please add a workflow so the PDF is posted in the pull request?

@MohamedLaghdafHABIBOULLAH
Copy link
Contributor Author

In fact, I tried it but it doesn’t work.

@MaxenceGollier
Copy link
Collaborator

can you rebase @MohamedLaghdafHABIBOULLAH ?

Copy link
Collaborator

@MaxenceGollier MaxenceGollier left a 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

Comment on lines +32 to +52
- 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
Copy link
Collaborator

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:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[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].
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- **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.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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.
Copy link
Collaborator

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)
Copy link
Collaborator

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 $$\nu$$ is not a kwarg for TR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants