-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
Folder/Directory descriptions not present #31443
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
Comments
Is it open? Can I take this PR? |
This is a great point. I’ve also found that navigating large codebases like scikit-learn can be challenging without brief descriptions or READMEs in subdirectories. Even a single sentence explaining the intent of each folder (e.g., algorithm families, utilities, tests) could go a long way for new contributors or those exploring the internals for the first time. It might be helpful to prioritize documentation for the most commonly accessed directories first (like sklearn/linear_model/, sklearn/ensemble/, etc.), and even include guidance for contributors who want to help write these descriptions. Definitely a solid suggestion that aligns well with improving dev onboarding and overall accessibility. |
Yeah exactly, that was the whole point. I am ready to work on this issue, if someone assigns me this |
The intent behind this sounds great, had similar experience starting up but since I work with scikit it was comparatively easier to navigate. That said, adding and maintaining separate It would also require a ton of maintenance I guess with every iteration. Would it make more sense to:
What do you think? |
Any kind of description about what files in which folders contain what, is doomed to be outdated really fast since we have quite a lot of contributions to the repo merged daily. However, I don't mind a generic overview of where things are in the Note that this is NOT a good first issue. Needs to be written by someone familiar with the codebase. Maybe @StefanieSenger wants to have a look. |
There are docstrings at the top of all the files, including the |
For the record, something I always repeat in our sprints, is along the lines of this answer: A generic overview of the files could be something like this: There are a few main categories of files in the repository:
|
Describe the issue linked to the documentation
I was navigating through the codebase, trying to find source code for some algorithms. I noticed that there are no descriptions of files present within a folder, which would actually make it easier to navigate through the codebase. We can have a small readme file within folders which would describe what is present in that folder.
Suggest a potential alternative/fix
No response
The text was updated successfully, but these errors were encountered: