Skip to content

Conversation

@MohamedLaghdafHABIBOULLAH
Copy link
Collaborator

@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

@codecov
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
Collaborator 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
Collaborator 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
Collaborator Author

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

Co-authored-by: Maxence Gollier <134112149+MaxenceGollier@users.noreply.github.com>
Co-authored-by: Maxence Gollier <134112149+MaxenceGollier@users.noreply.github.com>
@MaxenceGollier
Copy link
Collaborator

@MohamedLaghdafHABIBOULLAH can you rebase ?

- Removed old Benchmark.tex file and replaced it with a new version containing updated results.
- Consolidated benchmarking logic into a single Benchmark.jl file, removing separate benchmark-nnmf.jl and benchmark-svm.jl files.
- Updated comparison-config.jl to modify tolerance values and adjust configurations for benchmarking.
- Adjusted imports in Bench-utils.jl to remove unused dependencies and streamline the code.
- Updated Project.toml to include new dependencies for PrettyTables and ProximalAlgorithms.
- Modified the paper.md to correctly reference the new Benchmark.tex file location.
@MohamedLaghdafHABIBOULLAH
Copy link
Collaborator Author

MohamedLaghdafHABIBOULLAH commented Oct 3, 2025

Treated by 242

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.

4 participants