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

feat(status): add model status in statusbar tooltips and commandPalette, also adding heartbeat #3894

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Sma1lboy
Copy link
Collaborator

@Sma1lboy Sma1lboy commented Feb 24, 2025

  • Add a heartbeat every minute to obtain the model's health status
  • Add more details in the status bar tooltip
  • Add detailed status in the command panel
  • Use the isHealthCheckInProgress flag to prevent multiple health checks from occurring simultaneously, ensuring that each health check has a chance to complete
  • Ensure that subsequent requests are not lost through pendingConnectRequest, but are executed after the current health check is completed
  • Use debounceInterval to merge multiple connection requests within a short period, reducing unnecessary connection attempts
  • refactor Command Palette make it easily adding item in future

Status bar
IMG_8744

CommandPalette without code
image

CommandPalette with code
image

@Sma1lboy Sma1lboy marked this pull request as ready for review February 24, 2025 03:46
@Sma1lboy Sma1lboy requested a review from icycodes February 24, 2025 03:46
@Sma1lboy Sma1lboy changed the title feat(status): add model health status to status info and update tooltips feat(status): add model status in statusbar tooltips and commandPalette, also adding heartbeat Feb 28, 2025
@Sma1lboy Sma1lboy force-pushed the feat-adding-status-bar-hover-model-status branch from 82e670d to 2fec72d Compare February 28, 2025 03:36
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.

1 participant