Skip to content
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.

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

dev-minor #1509

Merged
merged 24 commits into from
Oct 30, 2024
Merged

dev-minor #1509

merged 24 commits into from
Oct 30, 2024

Conversation

emrgnt-cmplxty
Copy link
Contributor

@emrgnt-cmplxty emrgnt-cmplxty commented Oct 28, 2024


Important

This pull request adds new email providers, updates ingestion and vector search functionalities, and enhances prompt management with new tests and error handling.

  • Email Providers:
    • Add AsyncSMTPEmailProvider and ConsoleMockEmailProvider in email.py.
    • Update AuthProvider to integrate email providers.
  • Ingestion:
    • Replace ZeroxPDFParser with VLMPDFParser and BasicPDFParser in pdf_parser.py.
    • Update AudioParser, ImageParser, and other parsers to use new models and configurations.
    • Modify R2RIngestionProvider and UnstructuredIngestionProvider to support new parsers and models.
  • Vector Search:
    • Add support for binary vectors in vector.py.
    • Update create_index and upsert methods in vector.py to handle new quantization types.
  • Prompt Management:
    • Add new tests for prompt management in runner_sdk.py.
    • Improve error handling and access control for prompts.
  • Miscellaneous:
    • Remove deprecated zerox related code and configurations.
    • Update openapi.json to reflect API changes.
    • Minor refactoring and cleanup in various files.

This description was created by Ellipsis for 1d5d55a. It will automatically update as commits are pushed.

* Feature/tweak actions (#1507)

* up

* tweak actions

* adding bin sup and making it default

* tested and vetted
Copy link

vercel bot commented Oct 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
yc_demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 30, 2024 2:44pm
yc-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 30, 2024 2:44pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
recommendation_platform ⬜️ Ignored (Inspect) Oct 30, 2024 2:44pm

* up

* set verification to default false
* cleanup

* cleanup prompt mgmt

* up

* cleanup printout

* cleanup new parser logic, set vlm as default for all providers

* allow user to re-override
* Feature/tweak actions (#1507)

* up

* tweak actions

* modify exp backoff impl

---------

Co-authored-by: emrgnt-cmplxty <68796651+emrgnt-cmplxty@users.noreply.github.com>
* cleanup

* cleanup prompt mgmt

* up

* cleanup printout

* cleanup new parser logic, set vlm as default for all providers

* allow user to re-override

* add touchups
* add extra parsers

* add back ollama

* rvert auth workflow
* add extra parsers

* add prompt tests, cleanup

* add prompt tests, cleanup

* merge

* set mock console as default

* set mock console as default
* fix actions

* fallback

* import shutil
* fix actions

* fallback

* import shutil

* add basic pdf as extra parser in all configs
@emrgnt-cmplxty emrgnt-cmplxty marked this pull request as ready for review October 30, 2024 15:24
@emrgnt-cmplxty emrgnt-cmplxty merged commit ddfe870 into main Oct 30, 2024
35 of 49 checks passed
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 1d5d55a in 2 minutes and 33 seconds

More details
  • Looked at 5652 lines of code in 105 files
  • Skipped 2 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_B0DzK1kn5LTDM14j


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@NolanTrem NolanTrem deleted the dev-minor branch October 30, 2024 15:38
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.

3 participants