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

Improve documentation #2421

Closed
wants to merge 2 commits into from
Closed

Conversation

roaldarbol
Copy link

Hi guys! Hope this finds you well.
I've started a bunch tracking projects for myself and my colleagues recently, and especially the installation (on Windows, the usual CUDA/TF stuff) is a major stumbling point. So I thought I might as well update the documentation as I go along to. :-)

DeepLabCut has grown into an amazing ecosystem of tools, and I think the documentation has sort of grown equally to be a lot of smaller parts - and understandably so! But I think it would greatly improve the UX to streamline the documentation - give it an overhaul, and keep related information closer together.

For now, if you want to get an idea how I could imagine it looking, have a look at the Install with conda page in particular - I'm very happy to adapt if you have other ideas/visions of your own. But otherwise, I'm happy to just labour on over the coming months (have a tight deadline coming up in 3 weeks, so can spend more time on it afterwards)

Additionally, I've changed the _config.yml according to the default Jupyter Book file to expose all the options. Of course, I've kept the settings the same, it's more to make it easier for outsiders to contribute to the docs.

For now, leave the pull request open now matter what as I'm nowhere done with it.

PS From the default DLC installation I've needed to add some dependencies to the conda repo to be able to build the docs (didn't have e.g. jupyter-book). Do you have a special config file for development that contains these? Or do you have a list?

Cheers,
Mikkel

@MMathisLab MMathisLab self-requested a review October 24, 2023 13:23
@MMathisLab
Copy link
Member

hi @roaldarbol thanks for the contribution! A lot of changes, so it will take a while for me to work through it, but I agree our Windows in particular should be updated (none of us use windows regularly, as it's really a pain!).

I will note that we are in the middle of a big road-map listed overhaul of dropping tensorflow as well, so some bits about install will change there too.

@MMathisLab MMathisLab added enhancement New feature or request documentation documentation updates/comments labels Oct 24, 2023
@MMathisLab MMathisLab self-assigned this Oct 27, 2023
@roaldarbol
Copy link
Author

roaldarbol commented Nov 15, 2023

Hi @MMathisLab, sorry about the delayed response, suddenly had a lot of prep for a workshop. And no worries if you don't have time to review the PR currently, I get there will always be crunch periods, and my PR ended up being a bit messy, apologies! I haven't done too many PR requests before, so didn't have the review process in mind when I was typing away. :-)

The current bigger picture changes are as follows:

  • Landing page with sparse information
  • Unify installation docs in a single page
  • Expose all parameters in _config.yaml to make future changes easier

In terms of how to proceed from here on: Would you rather that we have a chat about what you would like from the docs, or should I churn on?
I am actively trying not to loose any information in the process, but for now they might change places and end up in different/new documents (that may or may not be included in the built site). I can revert to just having those changes and then make subsequent changes in new PRs if that'll be easier. Otherwise, I was thinking of running my changes in a separate version in my fork, so there would be an online version you could gauge the changes... maybe we can make a new_docs branch, and I can make PRs to that branch until further notice?

I am more than happy to help out testing stuff on Windows for you, especially from a UX perspective if that would be helpful! I usually run MacOS as my main, Windows on institutional devices (with better GPUs) and occasionally Raspberry Pi, though I hope no one's crazy(brave?) enough to try it there. ;-)

PS: Also really pleased to hear you'll be migrating away from TF, it just seems to be causing issues in the various libraries I've encountered it as a user.

@MMathisLab
Copy link
Member

MMathisLab commented Jun 5, 2024

thanks again for this! We are going to overall the docs a bit, but the soon-to-be-out PyTorch version is simplifying things; we make some adjustments here based on your feedback: #2495, but this means we will close this PR for now!

@MMathisLab MMathisLab closed this Jun 5, 2024
@roaldarbol
Copy link
Author

roaldarbol commented Jun 5, 2024

No worries, glad that it helped! I'd also be happy to provide some feedback on the new docs PR if you'd like an outsider's perspective. 😊

@MMathisLab
Copy link
Member

Absolutely 🥰🙏🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation documentation updates/comments enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants