feat: add brains fields snippet #3323
Merged
Conversation
This file contains 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
…3276) # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
🤖 I have created a release *beep* *boop* --- ## 0.0.316 (2024-09-30) ## What's Changed * fix(core): enforce langchain <0.3 for pydantic v1 by @StanGirard in #3217 * chore(main): release core 0.0.16 by @StanGirard in #3218 * feat(assistants): mock api by @StanGirard in #3195 * chore(examples): fix chainlit example by @StanGirard in #3223 * feat(integration): Notion by @chloedia in #3173 * fix(migration): order migrations by @StanGirard in #3231 * feat: remove n+1 query knowledge by @AmineDiro in #3238 * feat: introducing configurable retrieval workflows by @jacopo-chevallard in #3227 * fix: wrong default path for CHAT_LLM_CONFIG_PATH env variable by @jacopo-chevallard in #3247 * fix(frontend): sync folder for premium users by @Zewed in #3251 * fix: correctly passing the prompt provided by the user by @jacopo-chevallard in #3252 * fix: add sync user id aget_files by @AmineDiro in #3261 * feat: update sync_user.py to include sync user id in aget_files call by @StanGirard in #3276 * chore: update Dockerfile dependencies and copy files by @StanGirard in #3277 * chore: add wget to Dockerfile dependencies by @StanGirard in #3279 **Full Changelog**: v0.0.315...v0.0.316 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
The SyncsUser model in sync_models.py now has an optional status field. This change allows for more flexibility in managing user synchronization status. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
The syncNotion function is no longer needed in the ConnectionCards component, so it has been removed. This change simplifies the code and improves maintainability. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
🤖 I have created a release *beep* *boop* --- ## 0.0.317 (2024-09-30) ## What's Changed * feat: update SyncsUser status field to be optional by @StanGirard in #3284 * fix(frontend): onboarding modal by @Zewed in #3286 * feat: Remove syncNotion from ConnectionCards by @StanGirard in #3289 **Full Changelog**: v0.0.316...v0.0.317 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
This commit removes the syncNotion function from the ConnectionCards component. The function is no longer needed, simplifying the code and improving maintainability. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
🤖 I have created a release *beep* *boop* --- ## 0.0.318 (2024-09-30) ## What's Changed * feat: add syncNotion from ConnectionCards by @StanGirard in #3292 **Full Changelog**: v0.0.317...v0.0.318 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
#3298) The fetch_notion_pages function now includes error handling for rate-limited searches. If a rate-limited response is received, the function waits for 10 seconds before retrying the search. This ensures that the function can handle rate limits and continue fetching Notion pages. Co-authored-by: StanGirard <stan@example.com> Co-authored-by: ZeWed <zewed@example.com>
The syncNotion function is no longer needed in the ConnectionCards component, simplifying the code and improving maintainability. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
🤖 I have created a release *beep* *boop* --- ## 0.0.319 (2024-10-01) ## What's Changed * feat: Add error handling for rate-limited search in fetch_notion_pages by @StanGirard in #3298 * feat: Remove syncNotion from ConnectionCards by @StanGirard in #3300 **Full Changelog**: v0.0.318...v0.0.319 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Hello 👋 from Porter! Please merge this PR to finish setting up your application. Co-authored-by: porter-deployment-app[bot] <87230664+porter-deployment-app[bot]@users.noreply.github.com>
Hello 👋 from Porter! Please merge this PR to finish setting up your application. Co-authored-by: porter-deployment-app[bot] <87230664+porter-deployment-app[bot]@users.noreply.github.com>
# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate): --------- Co-authored-by: Zewed <dewez.antoine2@gmail.com>
# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
The image scale for OCR processing has been updated to improve accuracy. This change increases the scale from 300 to 500, resulting in better quality text recognition. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
The PDFGenerator class in `pdf_generator.py` has been updated to support markdown formatting when generating PDFs. This allows for better rendering of text with markdown syntax, improving the overall appearance of the generated PDFs. ```
# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
# Description - better logging for backend # Deployment: Set the following env variable ```bash USE_PARSEABLE=True PARSEABLE_URL= PARSEABLE_AUTH= ```
# Description - logging good, better
The PDFGenerator class in `pdf_generator.py` has been updated to include the logo in the header of the generated PDFs. This enhancement improves the visual appearance of the PDFs by adding branding elements. ``` # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
# Description - bettter logging --------- Co-authored-by: Stan Girard <girard.stanislas@gmail.com>
…3320) This commit updates the code in `celery_monitor.py` to handle task success events. When a task with a status of `SUCCESS` is received, it sends a notification with a success status and a corresponding description. Additionally, if the event has a `knowledge_id`, it updates the knowledge status to `UPLOADED`. This change improves the handling of task success events and ensures that notifications and knowledge statuses are properly updated. Note: This commit message follows the established convention of starting with a verb in the imperative form, followed by a brief description of the changes. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
The text retranscription instructions in the `DeadlyParser` class have been updated to provide more context and clarify the requirements. The new instructions include information about the type of document, the absence of illegal content or protected data, and the expectation to correct the entire text retranscription. This change improves the clarity and accuracy of the instructions for text retranscription. # Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dosubot
bot
added
size:XL
AmineDiro
deleted the
feature/core-216-fix-add-all-brains-field-in-entity
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description