-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
fix: Allow basic Astra DB Collection creation in combobox #5958
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
CodSpeed Performance ReportMerging #5958 will not alter performanceComparing Summary
|
codeflash-ai bot
added a commit
that referenced
this pull request
Jan 27, 2025
…by 11% in PR #5958 (`bugfix-collection-creation`) To optimize the provided Python program for speed, you can employ the following strategies. 1. Reduce the number of unnecessary dictionary accesses and lookups. 2. Simplify exception handling to avoid expensive operations. 3. Minimize repeated method calls by caching frequently accessed data. 4. Eliminate redundant code. Below is the optimized version of the program. ### Key Optimizations Made. 1. **Database Options initialization**. - Access the database list once and immediately transform it to the desired format. 2. **Collection Options initialization**. - Access `list_collections` once and use comprehensions directly to build the collection data. 3. **Update Build Configuration**. - Minimize dictionary accesses by caching frequently accessed sub-dictionaries like `api_endpoint` and `collection_name`. - Use dictionary methods like `update` for more efficient updates instead of multiple individual assignments. - Simplified condition checks and removed redundant code. These changes should help improve the speed of the program, especially in environments with a large number of database options and collections.
ogabrielluiz
approved these changes
Jan 28, 2025
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.
This pull request enables a combobox in the collection dropdown, which will allow a user to create a non-vectorize collection directly through langflow if desired.