Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@onuralpszr
Copy link
Collaborator

Introducing mkdocs-material to project and significant updates to the SAHI documentation and workflow configuration. It includes enhancements to the documentation structure, content, and presentation, as well as modifications to the workflow for dependency installation. Below is a summary of the most important changes grouped by theme:

Documentation Enhancements:

New Documentation Pages:

  • Added new documentation files for various SAHI components, such as annotation.md, auto_model.md, models/base.md, models/detectron2.md, models/huggingface.md, models/mmdet.md, models/roboflow.md, models/rtdetr.md, models/torchvision.md, models/ultralytics.md, models/yolov5.md, postprocess/combine.md, postprocess/utils.md, prediction.md, slicing.md, utils/coco.md, and utils/cv.md. Each file includes a reference to the respective module using the mkdocstrings syntax. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]

Documentation Restructuring:

  • Renamed docs/README.md to docs/index.md and updated its content to include a visually appealing introduction to SAHI, complete with badges, links, and a teaser image. The file now serves as the homepage for the documentation. [1] [2] [3] [4]

Configuration for MkDocs:

  • Added a mkdocs.yml configuration file to enable the use of MkDocs for generating SAHI documentation. This includes setting up navigation, themes, plugins, and markdown extensions to enhance the documentation's usability and appearance.

Workflow Configuration Updates:

  • Modified the dependency installation step in .github/workflows/mmdet.yml to use uv sync --group dev --extra mmdet instead of uv sync --extra dev --extra mmdet. This change ensures better grouping of dependencies during installation.

Adds netlify.toml for Netlify deployment configuration for test in PR.

These updates collectively improve the documentation's structure, accessibility, and functionality while refining the workflow setup for better compatibility.

@onuralpszr
Copy link
Collaborator Author

cc @fcakyon it is not completely done yet. But I wanted to post first initial PR so we can discuss further in PR to make it perfect. I had to some small adjustment in toml file for version and "override" package for pillow because mkdocs-material imaging and "inference" from roboflow have both different version choice so I am using small override to pass it. It won't effect page itself because I also moved all of them OUT of pypi extras, they won't be visible to anyone who installs from pypi (new pep standard landed a while ago so I wanted to get advantage of it.)

@fcakyon
Copy link
Collaborator

fcakyon commented Jul 21, 2025

@onuralpszr thanks for this awesome PR 🎉

I have pushed 2 updates, now all tests pass, all your suggestions make sense 💯

Is a free Netlify account enough for our case?

@onuralpszr onuralpszr force-pushed the docs/hello-mkdocs branch 2 times, most recently from f68ae33 to ce46414 Compare July 21, 2025 16:22
@onuralpszr
Copy link
Collaborator Author

@onuralpszr thanks for this awesome PR 🎉

I have pushed 2 updates, now all tests pass, all your suggestions make sense 💯

Is a free Netlify account enough for our case?

I am actually testing netlify config at my own PR to see and show you a preview of docs. Netlify has limits for how much usage but based on PR and commit of this repo it is %99 fine, I don't think you will broke the limit for a while (300 min build time is limit, I used on new project a bit much did not break it.)

@onuralpszr
Copy link
Collaborator Author

@fcakyon I made preview PR for myself to track docs and also here is preview link for to see docs preview :) Enjoy

https://docs-hello-mkdocs--obss-sahi-test-docs.netlify.app/
onuralpszr#1

@fcakyon
Copy link
Collaborator

fcakyon commented Jul 21, 2025

@fcakyon I made preview PR for myself to track docs and also here is preview link for to see docs preview :) Enjoy

docs-hello-mkdocs--obss-sahi-test-docs.netlify.app onuralpszr#1

very fast and responsive! even the notebooks look great 🔥

@onuralpszr
Copy link
Collaborator Author

onuralpszr commented Jul 21, 2025

@fcakyon I made preview PR for myself to track docs and also here is preview link for to see docs preview :) Enjoy
docs-hello-mkdocs--obss-sahi-test-docs.netlify.app onuralpszr#1

very fast and responsive! even the notebooks look great 🔥

Main deployment will be in github pages If you okay with it. One big question do you have "custom domain" or do you want to use what github gave us ? (That will be URL ıf I am not wrong https://obss.github.io/sahi/ )

@fcakyon
Copy link
Collaborator

fcakyon commented Jul 21, 2025

@onuralpszr we should remove notebooks from the demo folder to prevent duplication and update any reference to notebooks to the docs/notebooks folder.

@fcakyon
Copy link
Collaborator

fcakyon commented Jul 21, 2025

@onuralpszr we should remove notebooks from the demo folder to prevent duplication and update any reference to notebooks to the docs/notebooks folder.

GitHub pages default URL is definitely fine, no need for a custom domain 👍🏻

@onuralpszr
Copy link
Collaborator Author

@onuralpszr we should remove notebooks from the demo folder to prevent duplication and update any reference to notebooks to the docs/notebooks folder.

Okay I was going to ask for it. I will also?update all of the url around readme as well

@fcakyon
Copy link
Collaborator

fcakyon commented Jul 21, 2025

For future reference, please note that production documentation will be hosted on GitHub Pages. Additionally, PR documents will be deployed to Netlify for easier previewing. Can you confirm this, @onuralpszr?

@onuralpszr
Copy link
Collaborator Author

For future reference, please note that production documentation will be hosted on GitHub Pages. Additionally, PR documents will be deployed to Netlify for easier previewing. Can you confirm this, @onuralpszr?

Yes I can confirm 👍

@onuralpszr onuralpszr changed the title docs: 📝 Introduce mkdocs-material to sahi project docs: 📝 Introduce mkdocs-material to sahi project to have documentation page Jul 23, 2025
@fcakyon
Copy link
Collaborator

fcakyon commented Jul 24, 2025

@fcakyon, I’ve completed most of the work before the merge. I’ll also add a GitHub Page deployment action.

I have one question: would you prefer versioned documentation, such as ‘develop’ / ‘stable,’ or would you like to stick with a single version for now? It also depends on how you’d like to update the documentation it will naturally be more up-to-date than the ‘stable’ version If you keep single version. Typos and minor inconsistencies are acceptable, but breaking ABI changes would need to be handled differently.

Side note: I also noticed some mypy/class issues that I didn't want to address in this PR, but I’ll send a newer version in the next iteration.

@onuralpszr How about having a single version as you mentioned? We can have a dedicated md page giving details about breaking changes across versions.

@onuralpszr
Copy link
Collaborator Author

@fcakyon, I’ve completed most of the work before the merge. I’ll also add a GitHub Page deployment action.
I have one question: would you prefer versioned documentation, such as ‘develop’ / ‘stable,’ or would you like to stick with a single version for now? It also depends on how you’d like to update the documentation it will naturally be more up-to-date than the ‘stable’ version If you keep single version. Typos and minor inconsistencies are acceptable, but breaking ABI changes would need to be handled differently.
Side note: I also noticed some mypy/class issues that I didn't want to address in this PR, but I’ll send a newer version in the next iteration.

@onuralpszr How about having a single version as you mentioned? We can have a dedicated md page giving details about breaking changes across versions.

Okay I will be focusing on single version and I will add "CHANGELOG.md" so we can populate in time as well.

Thank you.

@fcakyon
Copy link
Collaborator

fcakyon commented Jul 24, 2025

@onuralpszr netlify app is also configured for the repo 🚀
CleanShot 2025-07-24 at 16 32 07@2x

@onuralpszr
Copy link
Collaborator Author

@fcakyon I include single version style for docs and I also added dependabot for "CI" I don't know your strategy yet about updating packages in pyproject.toml so I didn't not added yet. I only focused on docs and actions and CI first.

@onuralpszr
Copy link
Collaborator Author

@fcakyon after merge we also need to setup "gh-pages" as in settings->pages of repository enable "github packages and select "gh-pages" repo as documentation place and folder would be / (root)

@onuralpszr
Copy link
Collaborator Author

@onuralpszr
Copy link
Collaborator Author

Better Quick Start Installation Status

image

@onuralpszr
Copy link
Collaborator Author

Index page also updated (main infos got from paper)

https://deploy-preview-1--obss-sahi-test-docs.netlify.app/#what-is-sahi

@fcakyon fcakyon added this pull request to the merge queue Jul 24, 2025
Merged via the queue into obss:main with commit e857379 Jul 24, 2025
7 checks passed
fcakyon added a commit that referenced this pull request Jul 25, 2025
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