Skip to content

Conversation

@rshayer95
Copy link

Make the analyze process more verbose or add loading in CLI #24

Enhancements:

  • Added Timer for Process Execution: The analyzer CLI command now includes a timer that shows the elapsed time for task completion. This provides real-time feedback about the execution duration.

  • Improved User Feedback: The timer updates continuously on the same line during execution, reducing console clutter and offering a smoother experience. In case of errors or interruptions, the timer may appear on different lines temporarily.

  • Removed AgentChat_graph.json: This file, which was used solely for testing purposes, has been removed from the repository as it is no longer needed.

Description:

  • The timer feature enhances user experience by visually displaying the time it takes for the analyzer command to execute, allowing for more efficient tracking of task performance.
  • The AgentChat_graph.json file has been removed, as it was only required for testing and is no longer relevant to the project.

Copy link
Author

@rshayer95 rshayer95 left a comment

Choose a reason for hiding this comment

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

Removed Testing JSON input file.

@Aaditya-G Aaditya-G self-requested a review October 12, 2025 08:39
Copy link
Collaborator

@Aaditya-G Aaditya-G left a comment

Choose a reason for hiding this comment

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

Kindly address the changes @rshayer95

@rshayer95
Copy link
Author

Okay, will finalize the changes and submit back

- Fixed timer to display smoothly without spamming multiple lines
- Removed unnecessary comments for cleaner output
- Reverted .env.sample change to original placeholder value (OPENAI_API_KEY=your_openai_api_key)
@rshayer95
Copy link
Author

Made the fixes. Please review and let me know if anything missed. Otherwise, we are good to go

@Aaditya-G Aaditya-G self-requested a review October 17, 2025 03:28
Copy link
Collaborator

@Aaditya-G Aaditya-G left a comment

Choose a reason for hiding this comment

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

@rshayer95 the timer in your current implementation is not visible in the cli, can you please use tqdm instead?

@rshayer95
Copy link
Author

ok

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