Skip to content

UI Improvements and Thinking Model Configuration Enhancements#353

Merged
Llux-C merged 6 commits intodevelopfrom
feature/DefaultDisplay
May 13, 2025
Merged

UI Improvements and Thinking Model Configuration Enhancements#353
Llux-C merged 6 commits intodevelopfrom
feature/DefaultDisplay

Conversation

@kevin-mindverse
Copy link
Copy Markdown
Contributor

Overview

This PR includes multiple UI improvements and optimizations for the Thinking Model configuration, enhancing user experience and addressing potential issues with default values.

Changes

1. ThinkingModelModal Component Enhancement

  • Improved handling of Thinking Model configuration when backend returns null or undefined values
  • Set default model name to "deepseek-reasoner" and default API endpoint to "https://api.deepseek.com"
  • Ensured input fields show appropriate values regardless of backend response
  • Translated Chinese comments to English for better code internationalization

2. UI Text and Layout Improvements

  • Enhanced error message wording from "Cancel the current train" to "Cancel the current training run"
  • Reorganized the Thinking Model tooltip position for better readability
  • Added detailed explanation for Chain of Thought (CoT) functionality to help users understand its purpose

3. Default Configuration Updates

  • Modified Playground chat page default settings to disable L0 and L1 retrieval
  • Maintained system default temperature at 0.3 for more controlled outputs

4. Code Readability

  • Improved code structure with better handling of edge cases
  • Enhanced consistency in UI display and internal state management

5. User Experience Enhancement

  • Fixed validation issues when Thinking Model is not previously configured
  • Improved tooltips and explanations for advanced features

Testing

All changes have been verified to work correctly across relevant pages, particularly ensuring that the Thinking Model configuration properly handles various backend response scenarios.

@kevin-mindverse kevin-mindverse requested a review from Llux-C May 13, 2025 10:05
Copy link
Copy Markdown
Contributor

@Llux-C Llux-C left a comment

Choose a reason for hiding this comment

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

:)

@Llux-C Llux-C merged commit b4766a9 into develop May 13, 2025
1 check passed
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.

3 participants