Skip to content

Documentation revamp#102

Closed
BatyLeo wants to merge 9 commits into
mainfrom
revamp-doc
Closed

Documentation revamp#102
BatyLeo wants to merge 9 commits into
mainfrom
revamp-doc

Conversation

@BatyLeo

@BatyLeo BatyLeo commented Dec 12, 2023

Copy link
Copy Markdown
Member
  • Use DocStringExtensions.jl
  • Separate API between specific problems, losses, optim layers and internals
  • Background page for math (optim + ML), focus on discrete functions rather than linear programs
    • explain chain rules
    • add math details on each feature
  • Guide for choosing the right tool (as in https://scikit-learn.org/stable/tutorial/machine_learning_map/index.html)
  • Lots of tutorials (remove GridGraphs from the graph tuto)
    • remove grid graphs from the graphs tutorial
    • add a tutorial with JuMP
    • showcase all features in the tutorial
    • demonstrate use of kwargs

@BatyLeo BatyLeo added the documentation Improvements or additions to documentation label Dec 12, 2023
@BatyLeo BatyLeo self-assigned this Dec 12, 2023
@codecov-commenter

codecov-commenter commented Dec 12, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (5584bbe) 89.78% compared to head (20ff86e) 89.76%.
Report is 3 commits behind head on main.

Files Patch % Lines
src/perturbed/additive.jl 0.00% 2 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #102      +/-   ##
==========================================
- Coverage   89.78%   89.76%   -0.02%     
==========================================
  Files          24       24              
  Lines         695      694       -1     
==========================================
- Hits          624      623       -1     
  Misses         71       71              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rngil

rngil commented May 11, 2026

Copy link
Copy Markdown
Contributor

@BatyLeo
Is this still relevant?
We could split each checkbox into a separate issue/PR - should make it more manageable.

@BatyLeo

BatyLeo commented May 12, 2026

Copy link
Copy Markdown
Member Author

Most of it is still relevant, except maybe the training tutorials. Since we now have the new DFLBenchmarks.jl and DFLAlgorithms.jl, end-to-end training tutorials better belong in either one of those packages.

Most of these checkboxes already have their own issue I think.

@rngil

rngil commented May 12, 2026

Copy link
Copy Markdown
Contributor

Do you mind then if I use some of the things you already wrote here for separate issues/PRs?

@BatyLeo

BatyLeo commented May 13, 2026

Copy link
Copy Markdown
Member Author

Feel free to do so, the basics.jl tutorial might be interesting to expand for instance.

@gdalle

gdalle commented Jun 2, 2026

Copy link
Copy Markdown
Member

should we close this one ^^?

@BatyLeo

BatyLeo commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

Sure, this will never be merged as is

@BatyLeo BatyLeo closed this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs revamp Explain chain rules Better documentation Clarify & demonstrate use of kwargs

4 participants