Skip to content

README Roadmap and Intended PR Schedule#10

Open
etontackett wants to merge 43 commits into
mainfrom
docs/readme-roadmap
Open

README Roadmap and Intended PR Schedule#10
etontackett wants to merge 43 commits into
mainfrom
docs/readme-roadmap

Conversation

@etontackett

Copy link
Copy Markdown
Collaborator

Add project roadmap and anticipated pull request schedule to README

Description

This PR updates the README with a project roadmap which includes:
-Intended source code structure and organization
-Planned development sequence
-An expected PR schedule for future work

Motivation and Context

The goal of this PR is to make the project structure and development plan clear for contributors and reviewers.

Types of changes

  • CI
  • Docs
  • Feature
  • Fix
  • Performance
  • Refactor
  • Style
  • Test
  • Other (use sparingly):

Checklists:

Code and Comments
If this PR includes modifications to the code base, please select all that apply.

  • My code follows the code style of this project.
  • I have updated all package dependencies (if any).
  • I have included all relevant files to realize the functionality of the PR.
  • I have exported relevant functionality (if any).

API Documentation

  • For every exported function (if any), I have included a detailed docstring.
  • I have checked the spelling and grammar of all docstring updates through an external tool.
  • I have checked that the docstring's function signature is correctly formatted and has all arguments.
  • I have checked that the docstring's list of arguments, fields, or return values match the function.
  • I have compiled the docs locally and read through all docstring updates to check for errors.

Manual Documentation

  • I have checked the spelling and grammar of all manual updates through an external tool.
  • Any code included in the docstring is tested using doc tests to ensure consistency.
  • I have compiled the docs locally and read through all manual updates to check for errors.

Testing

  • I have added unit tests to cover my changes. (For Macros, be sure to check
    @code_lowered and
    @code_typed)
  • All new and existing tests passed.
  • I have achieved sufficient code coverage.

@vp314 vp314 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please think through what the file structure should look like. In the PR section, each PR should be its own subsection and there should be a table indicating what functionality/files are being implemented, their purpose, etc.

Comment thread README.md Outdated
Comment on lines +17 to +44
src/
RidgeRegression.jl

algorithms/
bidiagonalization.jl
closed_form.jl
gradient_descent.jl
stochastic_gradient_descent.jl

datasets.jl
design.jl

test/
runtests.jl

dataset_tests.jl
design_tests.jl
bidiagonalization_tests.jl
closed_form_tests.jl
gradient_descent_tests.jl
stochastic_gradient_descent_tests.jl

docs/
src/
index.md
design.md
api.md
```

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Use vertical bars and dashes to ensure spacing is correct

@vp314 vp314 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Minor changes

Comment thread README.md Outdated
Comment thread README.md Outdated

# Directory Structure

The source-code layout will be structured as such:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

"structured as follows:"

Comment thread README.md
├── src
│ ├── RidgeRegression.jl
│ ├── units.jl

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

treatments.jl
measurements.jl

Comment thread README.md Outdated
Comment on lines +32 to +42
│ ├── dataset_tests.jl
│ ├── encoding_tests.jl
│ ├── load_csv_dataset_tests.jl
│ ├── closed_form_tests.jl
│ ├── gradient_descent_tests.jl
│ ├── stochastic_gradient_descent_tests.jl
│ ├── compute_givens_test.jl
│ ├── rotate_rows_test.jl
│ ├── rotate_cols_test.jl
│ ├── apply_Ht_to_b_test.jl
│ └── bidiagonalize_with_H_test.jl

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This should be structured similarly to src/

So there should be a directory called src/

If you have multiple test files for a file in the src/ folder, then you can use a similar naming convention. For instance, if units.jl had two files "structs" and "onehotencode"

  • units_structs.jl
  • units_onehot_encode.jl

Comment thread README.md
Comment on lines +44 to +48
└── docs
├── make.jl
└── src
├── design.md
└── index.md

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Do some background research on documentation for software. Think about what makes the most sense for you.

Also think about the fact that you have an experiment that this software runs. So it should clearly indicate entry points to the code and exit points (e.g., tables, figures, datasets, etc.). So your documentation should also explain this as well.

For instance, "design.md" is not something you would find in standard software documentation but it is something you would find here.

etontackett and others added 2 commits June 29, 2026 13:30
Co-authored-by: Vivak Patel <vp314@users.noreply.github.com>
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