Skip to content

README update for PySide6 version#50

Open
C-Achard wants to merge 3 commits intocy/pre-release-fixes-2.0from
cy/pyside6-readme-update
Open

README update for PySide6 version#50
C-Achard wants to merge 3 commits intocy/pre-release-fixes-2.0from
cy/pyside6-readme-update

Conversation

@C-Achard
Copy link

Updates the previous README to reflect the new version of the GUI introduced in #35 and polished in #38.

Major rewrite of README to reflect the new PySide6-based DeepLabCut-Live-GUI. Updates include renamed project header and badges, updated license badge, new system requirements, detailed installation instructions (uv and conda/mamba workflows), run instructions, and a concise typical workflow (multi-camera preview, inference, and recording). Notes added about current limitations, references/citation, and contribution/feedback guidance; legacy tkinter-focused guidance and some older UI walkthrough text were removed or replaced with modernized guidance.
@C-Achard C-Achard self-assigned this Feb 17, 2026
@C-Achard C-Achard added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 17, 2026
@C-Achard C-Achard requested a review from Copilot February 17, 2026 08:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the README to reflect the modernized PySide6-based GUI introduced in PR #35 and refined in PR #38. The update transforms a detailed technical README into a concise, user-friendly quick-start guide that focuses on installation and basic workflows, while technical details have been moved to the docs/ folder.

Changes:

  • Replaced comprehensive technical documentation with streamlined installation and getting-started guide
  • Added support for both uv and conda installation workflows with inference backend selection
  • Updated to reflect PySide6 (Qt) interface with multi-camera support replacing tkinter
  • Corrected license reference (removed incorrect "Lesser" GPL claim, letting LICENSE file be authoritative)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@C-Achard C-Achard marked this pull request as draft February 17, 2026 10:05
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@C-Achard C-Achard marked this pull request as ready for review February 17, 2026 11:19
@C-Achard C-Achard requested a review from deruyter92 February 17, 2026 11:19
Copy link

@deruyter92 deruyter92 left a comment

Choose a reason for hiding this comment

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

Looks fine. Would only invert the order between cloning and installing.

Also: there is currently no reference for people who want to install from PyPI in in their (globally available) conda env instead of cloning & installing in their uv venv, but I'll leave it up to you if you want to include this.

Feel free to merge once you think it's ready!

Comment on lines +45 to 48
### Option A — Install with `uv`

#### 1) Create & activate a new virtual environment

Choose a reason for hiding this comment

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

Cloning should come before creatin venv

Comment on lines 49 to 57
```bash
brew install aravis
pip install pygobject
uv venv dlclivegui
# Linux/macOS:
source dlclivegui/bin/activate
# Windows (Command Prompt):
# .\dlclivegui\Scripts\activate.bat
# Windows (PowerShell):
# .\dlclivegui\Scripts\Activate.ps1
```

Choose a reason for hiding this comment

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

minor remark: adding newlines in between could improve readability

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 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments