PostHog client is initialized unconditionally during fish shell completion, adding ~10 seconds to every completion

Đang mở
  • Có gì không ổn?
#506 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
48/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
fish, python
Lĩnh vực
cli, performance

Hướng nghiên cứu

Bắt đầu bằng cách lần theo điểm vào của tính năng hoàn thành lệnh của fish và lời gọi lặp lại của lệnh comfy, sau đó theo dõi quá trình khởi động CLI đến nơi PostHog được khởi tạo. So sánh thời gian hoàn thành khi Analytics được bật và bị ngăn chặn; hoàn thành khi tính năng hoàn thành không còn thực hiện việc khởi tạo telemetry không cần thiết và độ trễ được báo cáo giảm xuống.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug

On my system, each completion invocation takes approximately 10.6 seconds because importing the CLI unconditionally initializes a PostHog background worker and completions invoke the comfy command twice. Suppressing PostHog reduces completion to roughly 0.8 seconds because the CLI is executed twice.

IDK if the fact that I have strict DNS blocking is what’s causing posthog to take the full 5s each time, but it seems possible

Therefore, there appear to be two related performance problems:

  1. Shell completion invokes the entire CLI twice per completion attempt.
  2. CLI startup eagerly initializes telemetry and other command infrastructure that completion does not need.

Analytics are disabled in my configuration, which makes the unconditional PostHog initialization additionally surprising.

Environment

  • comfy-cli: 1.12.0
  • Shell: Fish
  • OS: Linux
  • Analytics disabled in the Comfy configuration
  • DNS-level tracking-domain blocking is enabled

Reproduction

Compare an ordinary invocation of tabbing for a completion in fish shell then:

set -gx POSTHOG_API_KEY

It’s still kinda slow but it’s way more usable when you don’t post your hog each completion (~10s reduced to ~1s)

Ngôn ngữ chính
Python
Star
1k
Fork
176
Merge trung bình
1 ngày 11 giờ
Pull request đã merge (30 ngày)
87

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Comfy-Org/comfy-cli

Tất cả issue của Comfy-Org/comfy-cli

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.