Feat/24 more verbose analyze process#25
Open
rshayer95 wants to merge 8 commits intoRepello-AI:masterfrom
Open
Conversation
…e generating report
…e generating report
….com/rshayer95/Agent-Wiz into feat/24-more-verbose-analyze-process
rshayer95
commented
May 2, 2025
Author
rshayer95
left a comment
rshayer95
left a comment
There was a problem hiding this comment.
Removed Testing JSON input file.
Aaditya-G
requested changes
Oct 12, 2025
Collaborator
Aaditya-G
left a comment
Aaditya-G
left a comment
There was a problem hiding this comment.
Kindly address the changes @rshayer95
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)
Author
|
Made the fixes. Please review and let me know if anything missed. Otherwise, we are good to go |
Aaditya-G
requested changes
Oct 27, 2025
Collaborator
Aaditya-G
left a comment
Aaditya-G
left a comment
There was a problem hiding this comment.
@rshayer95 the timer in your current implementation is not visible in the cli, can you please use tqdm instead?
Author
|
ok |
Replaced custom timer with tqdm-based progress indicator to provide visible feedback during the analyze process. This improves user experience by displaying a dynamic progress bar and elapsed time in the CLI.
Author
|
@Aaditya-G Added tqdm for the timer, please let me know if it is working as expected and if anything is missing. If everything's working as expected, we are good to go. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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: