Skip to content

Added batch email fetching#18

Open
johntdavies wants to merge 2 commits into
AllAboutAI-YT:mainfrom
johntdavies:main
Open

Added batch email fetching#18
johntdavies wants to merge 2 commits into
AllAboutAI-YT:mainfrom
johntdavies:main

Conversation

@johntdavies

Copy link
Copy Markdown

I just added batch email fetching to speed things up and improved feedback with a progress bar.

…ded.

Added ability to use batching to search_and_process_emails() with a tqdm progress bar
Comment thread collect_emails.py
search_criteria = f'(SINCE "{start_date}" BEFORE "{end_date}")'
if search_keyword:
search_criteria += f' BODY "{search_keyword}"' # Ensure the correct combination of conditions
search_criteria += f' BODY "{search_keyword}"'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this line did not meaningfully change. I'd ask that you avoid the number of needless edits, as it complicates review.

Comment thread requirements.txt Outdated
PyPDF2
ollama
yaml
pyyaml

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

interested about why this change?

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