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

[Documentation] Refactor documentation #257

Merged
merged 8 commits into from
Dec 10, 2024

Conversation

lucas-diedrich
Copy link
Contributor

@lucas-diedrich lucas-diedrich commented Dec 3, 2024

Restructure alphabase documentation

This pull request

  • Adds installation instructions ([...]/docs/installation.rst)

  • Adds a API documentation for all modules ([...]/docs/api.rst)

  • Adds references ([...]/docs/references.rst)

  • Restructures index page (add links to installation, API documentation, notebooks, references), adds description + reference.

Index

Old

OLD-INDEX

New

NEW-INDEX

API

New

API

Copy link
Contributor

@mschwoer mschwoer left a comment

Choose a reason for hiding this comment

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

LGTM

docs/installation.rst Outdated Show resolved Hide resolved
@@ -0,0 +1,41 @@
Installation
Copy link
Contributor

Choose a reason for hiding this comment

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

This now has some redundancies with the Readme.md. If you want to follow-up, you can have a look at the structure of https://github.com/MannLabs/alphadia?tab=readme-ov-file#installation where we had the same problem.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will address this in another pull request 👍

docs/installation.rst Outdated Show resolved Hide resolved
docs/installation.rst Outdated Show resolved Hide resolved
…etup of conda environment, updated installation instructions for development version
@lucas-diedrich lucas-diedrich merged commit 1a2e0e0 into MannLabs:main Dec 10, 2024
10 checks passed
@lucas-diedrich lucas-diedrich deleted the refactor_docs_I branch December 10, 2024 07:20
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.

2 participants