Skip to content
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

Unify ecosystem.md #1923

Merged
merged 11 commits into from
May 15, 2022

Conversation

Saransh-cpp
Copy link
Member

Closes #1922

Added all the packages from the website to the docs!

PR Checklist

  • Tests are added
  • Entry in NEWS.md
  • Documentation, if applicable

@codecov-commenter
Copy link

codecov-commenter commented Apr 1, 2022

Codecov Report

Merging #1923 (50b455b) into master (57beb23) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1923   +/-   ##
=======================================
  Coverage   86.43%   86.43%           
=======================================
  Files          18       18           
  Lines        1430     1430           
=======================================
  Hits         1236     1236           
  Misses        194      194           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57beb23...50b455b. Read the comment docs.

docs/src/ecosystem.md Outdated Show resolved Hide resolved
docs/src/ecosystem.md Show resolved Hide resolved
Co-authored-by: Kyle Daruwalla <[email protected]>
@Saransh-cpp
Copy link
Member Author

Thank you for all the suggestions! Should I also add NNlib.jl, Functors.jl, Optimisers.jl, and DaggerFlux.jl?

docs/src/ecosystem.md Outdated Show resolved Hide resolved
docs/src/ecosystem.md Outdated Show resolved Hide resolved
docs/src/ecosystem.md Outdated Show resolved Hide resolved
docs/src/ecosystem.md Outdated Show resolved Hide resolved
Comment on lines 66 to 77
## Parameters

- [Parameters.jl](https://github.com/mauro3/Parameters.jl) types with default field values, keyword constructors and (un-)pack macros.
- [ParameterSchedulers.jl](https://github.com/darsnack/ParameterSchedulers.jl) standard scheduling policies for machine learning.

## Statistics

- [OnlineStats.jl](https://github.com/joshday/OnlineStats.jl) provides single-pass algorithms for statistics.

## Time series

- [FluxArchitectures.jl](https://github.com/sdobber/FluxArchitectures.jl) is a collection of advanced network architectures for time series forecasting.
Copy link
Member

Choose a reason for hiding this comment

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

Nice to have section headings, but perhaps they could be better ordered? Making rough groups, ordered by how closely associated they are to Flux:

  • Actual Flux models: Vision / Language / Graph / Reinforcement learning / Time series
  • Tools closely associated with Flux, maybe defined as things you're unlikely to have met if you never used Flux: Datasets / FastAI.jl / Plumbing / ParameterSchedulers.jl
  • Differentiable but not necc. ML tools, like SciML, Probabilistic programming
  • Useful things which aren't domain-specific at all: ArgParse.jl, ProgressMeter.jl, DataFrames.jl

Maybe they even want another level of headings, not sure. Or maybe there should be only 4-5 H2 headings, but some text between bullet point lists, e.g. under ## Flux models lists Vision, NLP, RL, etc. ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, grouping these headings under broader H2 headings makes sense. This would definitely make it easier to navigate through the page. I'll add them!

Copy link
Member Author

Choose a reason for hiding this comment

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

image

Definitely makes the in-website navigation much better!

@CarloLucibello CarloLucibello merged commit 944fd43 into FluxML:master May 15, 2022
@Saransh-cpp Saransh-cpp deleted the inconsistent-ecosystem-docs branch May 15, 2022 06:31
@arcAman07 arcAman07 mentioned this pull request May 15, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistent "Julia ecosystem" docs
6 participants