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

chore(main): release 0.0.231 #2434

Conversation

StanGirard
Copy link
Collaborator

@StanGirard StanGirard commented Apr 16, 2024

🤖 I have created a release beep boop

0.0.231 (2024-04-19)

What's Changed

Full Changelog: v0.0.230...v0.0.231


This PR was generated with Release Please. See documentation.


🚀 This description was created by Ellipsis for commit a16fa4d

Summary:

This PR releases version 0.0.231, introducing several feature enhancements and bug fixes across the assistant, analytics, Docker, and frontend modules.

Key points:

  • Release version 0.0.231 with feature enhancements and bug fixes across multiple modules
  • User usage update and pricing calculation added to ITO assistant
  • Improved prompt summary in assistant module
  • PDF generation functionality and email enhancements added
  • Analytics optimized by relying on SQL instead of Python loop
  • Token output limit fixed in assistant summary
  • Docker build commands and dependencies updated
  • DEFAULT_DOCUMENT_PROMPT in quivr_rag.py refactored
  • Porter Applications for quivr-back and quivr-demo-front enabled
  • Null brain id issue fixed in assistants module
  • Prompt improved for better insights in summary module
  • CPU and memory configurations for AWS task definitions updated
  • Quivr Assistants added in frontend

Generated with ❤️ by ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 9aa6637 to 80db30a Compare April 16, 2024 16:15
Copy link

vercel bot commented Apr 16, 2024

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

Name Status Preview Comments Updated (UTC)
quivrapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 19, 2024 8:39am

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. autorelease: tagged labels Apr 16, 2024
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 the entire pull request up to 80db30a
  • Looked at 19 lines of code in 1 files
  • Took 43 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    The changelog update looks correct, reflecting the new version and the changes made. Good job on keeping the changelog up to date.
  • Reasoning:
    This PR is a release PR, which is automatically generated. The changes are only in the CHANGELOG.md file, reflecting the new version and the changes made in the new version. There is no actual code change in this PR, so there's no code to review for bugs or best practices. The only thing to check is whether the changelog correctly reflects the changes made in the new version.

Workflow ID: wflow_z1GSo3X96khF0Hv7


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

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!

  • Performed an incremental review on 26235bd
  • Looked at 19 lines of code in 1 files
  • Took 2 minutes and 4 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 100%
  • Grade: 0%
  • Comment:
    The index before and after the change is the same, which is not possible if there were changes made to the file. Please check the diff generation process.
  • Reasoning:
    This is a release PR, and it seems to be auto-generated. The changes are only in the CHANGELOG.md file, and they seem to be correctly reflecting the changes made in the mentioned PRs. However, there seems to be a mistake in the diff. The index before and after the change is the same, which is not possible if there were changes made to the file.

Workflow ID: wflow_P41iBoqhAvRus12o


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 26235bd to 8790166 Compare April 17, 2024 13:01
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Apr 17, 2024
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!

  • Performed an incremental review on 8790166
  • Looked at 20 lines of code in 1 files
  • Took 1 minute and 4 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    The changelog update seems to be in order, with the correct version number and date. The included changes and their respective PR links are also correctly formatted and informative.
  • Reasoning:
    This is a release PR, and it seems to be auto-generated. The changes in the PR are only in the CHANGELOG.md file, reflecting the new version and the features added. There doesn't seem to be any code changes in this PR, so there's no code to review for bugs or best practices. However, I'll check the formatting and information in the CHANGELOG.md file.

Workflow ID: wflow_IiropksMQGDdoGiM


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 8790166 to 91756da Compare April 17, 2024 14:24
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!

  • Performed an incremental review on 91756da
  • Looked at 21 lines of code in 1 files
  • Took 1 minute and 39 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:9:
  • Assessed confidence : 33%
  • Comment:
    There's a small typo in the description of the change made in PR 2437. It should be 'rather than python loop', not 'rather that python loop'.
  • Reasoning:
    This PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the date correctly updated. The list of changes also seems to be in order, with the correct PR links and descriptions. However, there's a typo in the description of one of the changes. It says 'rather that python loop' instead of 'rather than python loop'. This is a minor issue, but it's worth pointing out for the sake of clarity and professionalism in the documentation.

Workflow ID: wflow_YwQGibdFrn0Xi4kH


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 91756da to 8f584e6 Compare April 17, 2024 15:34
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!

  • Performed an incremental review on 8f584e6
  • Looked at 22 lines of code in 1 files
  • Took 1 minute and 32 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    The changes in the CHANGELOG.md file are correctly formatted and informative. Good job!
  • Reasoning:
    This is a release PR, and it seems to be auto-generated. The changes are only in the CHANGELOG.md file, and they seem to be correctly formatted and informative. There's no code to review here, so there's no chance of bugs, performance issues, or security issues. The PR description is also clear and informative.

Workflow ID: wflow_I46J4EggbPUABmoV


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 2 days remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

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!

  • Performed an incremental review on 8dfdee0
  • Looked at 23 lines of code in 1 files
  • Took 1 minute and 53 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:8:
  • Assessed confidence : 100%
  • Grade: 0%
  • Comment:
    There's a typo in the description of the change made in PR 2437. It should be 'rather than python loop', not 'rather that python loop'.
  • Reasoning:
    The PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the date correctly updated. The list of changes also seems to be in order, with the correct PR links and descriptions. However, there's a typo in one of the descriptions: 'rather that python loop' should be 'rather than python loop'.

Workflow ID: wflow_2sCUAZVN9Px41ixx


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 8dfdee0 to 4a56f58 Compare April 17, 2024 21:38
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!

  • Performed an incremental review on 4a56f58
  • Looked at 24 lines of code in 1 files
  • Took 2 minutes and 32 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 100%
  • Grade: 0%
  • Comment:
    The index of the file before and after the changes is the same. This could indicate that the file hasn't actually been changed. Please double-check this.
  • Reasoning:
    This PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the list of changes. However, the diff shows that the index of the file before and after the changes is the same, which is unusual. This could be a mistake in the diff generation, or it could mean that the file hasn't actually been changed. I need to comment on this to bring it to the attention of the PR author.

Workflow ID: wflow_hzfSlV3sgz64nPtI


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 4a56f58 to 897976b Compare April 17, 2024 22:27
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!

  • Performed an incremental review on 897976b
  • Looked at 25 lines of code in 1 files
  • Took 1 minute and 18 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 50%
  • Comment:
    The index before and after the change is the same. This could be a mistake or a bug in the tool that generated the PR. Please double-check.
  • Reasoning:
    This PR is a release PR, and it's updating the CHANGELOG.md file. The changes seem to be in order, with the new version number and the list of changes. However, the diff shows that the index before and after the change is the same, which is unusual. This could be a mistake or a bug in the tool that generated the PR.

Workflow ID: wflow_7zLMbd1cRaT4Y4o4


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 897976b to da2e26e Compare April 18, 2024 15:56
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!

  • Performed an incremental review on da2e26e
  • Looked at 26 lines of code in 1 files
  • Took 1 minute and 44 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    The changes in the CHANGELOG.md file look good. They are well-formatted and include the correct information about the changes in the new version.
  • Reasoning:
    This PR is a release PR, which is mostly about updating the CHANGELOG.md file. The diff shows that the changes are only in the CHANGELOG.md file, and they seem to be correctly formatted and include the correct information about the changes in the new version. There are no code changes to review, so there's no need to check for bugs or violations of best practices. The PR description is also well-written and includes all the necessary information.

Workflow ID: wflow_5AD7yhIxzrpCmGuj


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 1 day remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from da2e26e to 299d978 Compare April 18, 2024 16:24
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.

❌ Changes requested.

  • Performed an incremental review on 299d978
  • Looked at 27 lines of code in 1 files
  • Took 1 minute and 50 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 0 additional comments because they didn't meet confidence threshold of 50%.

Workflow ID: wflow_KDmg0HRLxxcnCkIe


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. We'll respond in a few minutes. Learn more here.

⌛ You have 23 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

## What's Changed
* feat(assistants): Add user usage update and pricing calculation to ITO assistant by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2433
* feat(assistant): improve prompt summary by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2435
* feat(assistants): Add PDF generation functionality and nice emails by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2436
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a small typo in the changelog entry. It should be 'rather than' instead of 'rather that'.

Suggested change
* feat(assistants): Add PDF generation functionality and nice emails by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2436
* feat(analytics): rely on sql rather than python loop for brains by @StanGirard in https://github.com/QuivrHQ/quivr/pull/2437

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 299d978 to 9ebbcc3 Compare April 18, 2024 21:04
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!

  • Performed an incremental review on 9ebbcc3
  • Looked at 29 lines of code in 1 files
  • Took 7 minutes and 34 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    This PR seems to be auto-generated for a new release. It only includes changes to the CHANGELOG.md file, and the PR description provides a clear summary of the changes included in this release. There are no code changes to review in this PR.
  • Reasoning:
    This is a release PR, and it seems to be auto-generated. The changes in the PR are only in the CHANGELOG.md file, reflecting the new version and the changes made. There doesn't seem to be any code changes in this PR, so there's no code to review for bugs or best practices. The PR description is clear and provides links to all the changes included in this release.

Workflow ID: wflow_TNxFGoO1Cb9LSB1R


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 18 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 9ebbcc3 to 5c421ee Compare April 18, 2024 22:18
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!

  • Performed an incremental review on 5c421ee
  • Looked at 30 lines of code in 1 files
  • Took 4 minutes and 4 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    The version number seems to be incremented correctly, and the changes listed match the changes made in the commits. No issues found.
  • Reasoning:
    This is a release PR, and it seems to be auto-generated. The changes in the PR are only in the CHANGELOG.md file, reflecting the new version and the changes made. There is no actual code change in this PR, so there's no code to review for bugs or violations. However, I'll check if the version number is incremented correctly and if the changes listed match the changes made in the commits.

Workflow ID: wflow_lM7OlqeTgDRfTKBV


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 17 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard force-pushed the release-please--branches--main--components--release-please-action branch from 5c421ee to a16fa4d Compare April 19, 2024 08:37
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!

  • Performed an incremental review on a16fa4d
  • Looked at 31 lines of code in 1 files
  • Took 1 minute and 48 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. CHANGELOG.md:1:
  • Assessed confidence : 0%
  • Comment:
    The changes in the CHANGELOG.md file correctly reflect the updates made in this version. No issues found.
  • Reasoning:
    This is a release PR, and the changes are only in the CHANGELOG.md file. The changes seem to be correctly reflecting the updates made in the version. There are no code changes to review here.

Workflow ID: wflow_j7mwwvN8n5T4WSUO


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

⌛ You have 7 hours remaining in your free trial. Upgrade at https://app.ellipsis.dev for $20/developer/month. If you have any questions, reach us at help@ellipsis.dev

@StanGirard StanGirard merged commit 08c857a into main Apr 19, 2024
6 checks passed
@StanGirard
Copy link
Collaborator Author

🤖 Release is at https://github.com/QuivrHQ/quivr/releases/tag/v0.0.231 🌻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autorelease: tagged size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant