-
Notifications
You must be signed in to change notification settings - Fork 252
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 #1148
Dev minor #1148
Conversation
* improve telemetry * finish telemetry tweaks
* improve telemetry * finish telemetry tweaks * up
* improve telemetry * finish telemetry tweaks * up * fallback to main
* add system diagram * rm multi * fix overview * cleanup and fix * fix syntax
* up * up * up * up
* cleanups * bump pkg
* rm unstructured tag, add auto build on dev and dev-minor * delete old Dockerfile * add retries
* adding test pypi * make unstructured docker file * add on-push effects for dev * tweak workflows
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
* adding test pypi * make unstructured docker file * add on-push effects for dev * tweak workflows
* add boto3 * explain feature add
* update prompt * test and fix. Add dockerfile.dev * up
There was a problem hiding this 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 2d460c9 in 48 seconds
More details
- Looked at
2389
lines of code in30
files - Skipped
8
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. py/core/providers/prompts/r2r_prompts.py:199
- Draft comment:
Consider documenting the choice ofuuid5
withNAMESPACE_DNS
for generating prompt IDs. This ensures consistency but might need explanation for future maintainers. - Reason this comment was not posted:
Confidence changes required:50%
The use ofuuid5
withNAMESPACE_DNS
for generating prompt IDs is a good choice for consistency, but it should be documented or explained why this specific method is chosen over others.
Workflow ID: wflow_WXZmloVtgp1LAu9G
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
There was a problem hiding this 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! Incremental review on 198f872 in 49 seconds
More details
- Looked at
13
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_E6S34t6w5ySgJhly
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
@@ -185,6 +185,7 @@ async def parse( | |||
**self.config.chunking_config.extra_fields, | |||
) | |||
|
|||
iteration = 0 # if there are no chunks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we not be failing if a document has no chunks?
feat: add Docker build workflow and enhance prompt management
Summary:
Add Docker build workflow, enhance prompt management, and update file types and configurations.
Key points:
build-docker.yml
for Docker image build and publish ondev
anddev-minor
branches; removebuild-main.yml
and updatepublish-to-pypi.yml
.Dockerfile
andDockerfile.dev
withtesseract-ocr
andlibreoffice
; add Unstructured models initialization.created_at
andupdated_at
timestamps toPrompt
inr2r_prompts.py
; update prompt loading logic.parsing_and_chunking.mdx
anddocument.py
to includeBMP
,EPUB
,HEIC
, etc.auto
inr2r.toml
; add line item reference reminder inrag_agent.yaml
.Generated with ❤️ by ellipsis.dev