Skip to content

Conversation

@qchiujunhao
Copy link
Contributor

  1. changed to NiceGUI
  2. use DSPy

Previously, when refreshing the app on the Bookmarks page, it would show 'No bookmarks available' even if bookmarks existed. This happened because the main Chat page didn’t run and therefore didn’t load the chat history. This issue is now fixed.
@qchiujunhao qchiujunhao requested a review from Copilot June 14, 2025 01:36
Copy link
Contributor

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 PR updates the tool to version 0.3.0, switches from Streamlit to NiceGUI with DSPy integration, and adjusts configurations like port numbers and dependency requirements.

  • Update tool version and container image in chat_analysis.xml
  • Change port and dependency references in Dockerfile
  • Revise bookmark handling and chat history saving in Bookmarks.py and Chat.py

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/chat_analysis.xml Updated tool version and container image reference
pages/1_Bookmarks.py Added bookmark JSON persistence and refined UI logic
Dockerfile Changed dependency file and exposed new port
Chat.py Updated summary stats logic, revised prompts, removed unused UI blocks
.dockerignore Added new ignore patterns for test files and outputs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@qchiujunhao qchiujunhao merged commit ca6d649 into smolagents-release Jun 14, 2025
8 checks 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.

2 participants