feat: Add tqdm progress bars for PDF parsing and summary generation#19
feat: Add tqdm progress bars for PDF parsing and summary generation#19blueprintparadise wants to merge 2 commits intoVectifyAI:mainfrom
Conversation
|
Hi @mxyhi, thanks for the contribution! The idea of adding progress bars is great for CLI users. However, there are some issues that need to be addressed before this can be merged. Critical: Based on outdated code (will crash on merge) The PR modifies a call to Design: Avoid duplicating existing logic Rather than re-implementing the summary generation flow at the call site, please consider adding a Minor issues
Happy to see this landed once the rebase and design issues are resolved! |
Hey guys, absolutely loved your project. I have been playing with it for the last few days and it shows a lot of promise.
Here is my first pull request. It just adds a progress bar to the cli output. Please let me know if you face any issue during integration.
Regards,
RSH