FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix entrypoint - Requested changes by maximpavliv · Pull Request #2883 · DeepLabCut/DeepLabCut · GitHub

/ DeepLabCut Public

Fix entrypoint - Requested changes - #2883

Merged
n-poulsen merged 1 commit into
pytorch_dlcfrom
maxim/fix_entrypoint
Feb 17, 2025
ConversationCommits (1)ChecksFiles changed
Merged

Fix entrypoint - Requested changes#2883
n-poulsen merged 1 commit into
pytorch_dlcfrom
maxim/fix_entrypoint

Conversation

maximpavliv commented Feb 17, 2025

Copy link
Copy Markdown
Contributor

Following #2863:

  • Change CLI command name from dlc to deeplabcut
  • Create main() function in deeplabcut/__main__.py (somehow in fix entrypoint for dlc #2863 it was working, but I'm not sure why, as deeplabcut.__main__:main is calling the main() function, which didn't exist).

n-poulsen approved these changes Feb 17, 2025
View reviewed changes

n-poulsen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM!

n-poulsen merged commit a31cca5 into pytorch_dlc Feb 17, 2025
maximpavliv mentioned this pull request Feb 17, 2025
Merged
MMathisLab deleted the maxim/fix_entrypoint branch April 14, 2025 11:48
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
n-poulsen n-poulsen approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

DLC3.0🔥 Installation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL