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

Reorganize documentation pages #8325

Merged
merged 5 commits into from
Jul 5, 2024
Merged

Conversation

ngxson
Copy link
Collaborator

@ngxson ngxson commented Jul 5, 2024

This PR aims to make documentation easier for new users / new contributors:

  • Correct usage for markdown headers. All headers must now follow order: h1 #, h2 ##, h3 ###,...
  • Make "basic usage" more visible
  • Remove some outdated info
  • Break docs into small pages ==> The main README has less info now, mostly links to more technical pages

Page structure:

  • README.md ==> main page
  • docs
    • backend
      • SYCL
      • BLIS
    • development ==> various small technical docs (probably outdated?)
    • build ==> how to build
    • docker ==> specific page for Docker-related info
    • install ==> how to install pre-built binary
    • android ==> How to build on android (probably can be merged to "build" page)

To preview the change, go to: https://github.com/ngxson/llama.cpp/tree/xsn/reorganize_docs

With correct markdown usage, TOC can now be displayed correctly on github. We can add a small script to generate TOC in the future:

image


@ngxson ngxson requested a review from ggerganov July 5, 2024 10:06
@github-actions github-actions bot added documentation Improvements or additions to documentation examples labels Jul 5, 2024
@ngxson ngxson merged commit be20e7f into ggerganov:master Jul 5, 2024
7 checks passed
@NeoZhangJianyu
Copy link
Collaborator

@ngxson
It's OK to refactor the documents.

But I don't find any link of README-sycl.md in README.md.
And there is no any description of backend types in README.md.

Through the online document will be easy to read, *.md is still important document to developer and users.

So, I suggest to add a chapter in REAMDE.md for backend types and include the links to related documents.

@ngxson
Copy link
Collaborator Author

ngxson commented Jul 6, 2024

@NeoZhangJianyu The link to SYCL has been moved to docs/build.md, but I agree that this make these pages less visible.

So, I suggest to add a chapter in REAMDE.md for backend types and include the links to related documents.

Sounds good to me, I'll that this section in my next PR

@NeoZhangJianyu
Copy link
Collaborator

@ngxson
It's great!

arthw pushed a commit to arthw/llama.cpp that referenced this pull request Jul 13, 2024
* re-organize docs

* add link among docs

* add link to build docs

* fix style

* de-duplicate sections
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Jul 13, 2024
* re-organize docs

* add link among docs

* add link to build docs

* fix style

* de-duplicate sections
NikolaiLyssogor added a commit to NikolaiLyssogor/llama.cpp that referenced this pull request Jul 14, 2024
Fixes a few links to within the repo that were broken in the reorganization of the
documentation in ggerganov#8325.
ggerganov pushed a commit that referenced this pull request Jul 15, 2024
Fixes a few links to within the repo that were broken in the reorganization of the
documentation in #8325.
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Jul 27, 2024
Fixes a few links to within the repo that were broken in the reorganization of the
documentation in ggerganov#8325.
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 examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants