Skip to content

Fix/thinking toolkit warning #3155

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Fix/thinking toolkit warning #3155

wants to merge 2 commits into from

Conversation

@shudanluo
Copy link

Summary

Fix verbose and confusing warnings in workforce initialization that were causing user confusion.

Problem

Issue #2820 reported "Unknown Warning" when using ThinkingToolkit with ChatAgent. The warnings were:

  • Overly verbose (multi-line messages)
  • Using inappropriate warning level for informational content
  • Causing confusion for users

Solution

  • Replaced logger.warning() with logger.info() for configuration guidance
  • Simplified messages while maintaining helpful information
  • Improved user experience by reducing noise in logs

Changes

  • Modified 3 warning messages in camel/societies/workforce/workforce.py
  • Reduced 27 lines of verbose warnings to 6 lines of concise info messages

Testing

  • Code compiles without errors
  • Maintains all functionality while improving UX

@shudanluo
feat: Implement LLM-supervise deduplication strategy
70ef4f9 
- Add _deduplicate_with_llm_supervision function
- Add _llm_judge_duplicates helper function
- Fix circular import issues
- Add comprehensive test suite with 9 test cases
- Maintain backward compatibility with existing code
@shudanluo
Fix verbose warnings in workforce initialization
97ee990 
- Replace lengthy logger.warning() calls with concise logger.info() messages
- Improve user experience by reducing confusing warning output
- Maintain helpful guidance for configuration options
- Addresses issue camel-ai#2820: Unknown Warning when Using ThinkingToolkit with ChatAgent

The previous warnings were overly verbose and used inappropriate warning level
for informational configuration guidance.
@nitpicker55555 nitpicker55555 marked this pull request as draft September 18, 2025 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@shudanluo