fix: General Fixes and Improvements from v1.0.19.2 Fixes Branch #4411
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
Add opensearch-py dependency to pyproject.toml
♻️ (RetrieverTool.py): refactor build method signature to accept additional keyword arguments for future extensibility
* 🐛 (base.py): fix AttributeError by checking if custom_component has set_event_manager method before calling it * 📝 (base.py): Import Component from langflow.custom to improve code readability and maintainability ♻️ (base.py): Refactor code to use isinstance() method for checking if custom_component is an instance of Component
…y_fix_release (#4208) Refactor: Eliminate Global Variables for Improved Code Maintainability - Replaced global variables with local variables or class attributes. - Enhanced code readability and reduced potential side effects.
update example
#4203) fix: avoids error NameError: name 'MAX_NUMBER_OF_FIELDS' is not defined and fixes build method Co-authored-by: Edwin Jose <[email protected]>
#4230) 🔧 (SQLDatabase.py): update build method signature to accept additional keyword arguments for future extensibility 🔧 (SQLExecutor.py): update method signature to accept additional keyword arguments for future extensibility
Adds a dockerfile with an entrypoint for use with Datastax Langflow
…uthentication 🔧 (api.tsx): Replace universal-cookie import with react-cookie for consistency 🔧 (authStore.ts): Replace universal-cookie import with react-cookie for consistency 🔧 (use-get-global-variables.ts): Add check to only fetch global variables if user is authenticated ✨ (use-get-mutation-global-variables.ts): Add mutation function to fetch and update global variables 🔧 (authStore.ts): Replace universal-cookie import with react-cookie for consistency
dosubot
bot
added
the
size:XL
Cristhianzl
changed the title
Cristhianzl
changed the title
CodSpeed Performance ReportMerging #4411 will degrade performances by 16.71%Comparing Summary
Benchmarks breakdown
|
|
LGTM |
… with project requirements and dependencies.
…0.0.99 to 0.0.97 to resolve compatibility issues
… 0.3.12 to resolve compatibility issues with dependencies
…99 to utilize the latest features and improvements
Cristhianzl
requested review from
edwinjosechittilappilly and
jordanrfrazier
and removed request for
jordanrfrazier and
edwinjosechittilappilly
italojohnny
approved these changes
Nov 6, 2024
jordanrfrazier
approved these changes
Nov 6, 2024
joaoguilhermeS
added a commit
that referenced
this pull request
Nov 7, 2024
* fix: update assistants client import (#4150) * remove unnecessary patch * remove unnecessary patch * compatible release operator * chore: add opensearch-py dependency (#4134) Add opensearch-py dependency to pyproject.toml * patch version * lock * lock some packages to speed up pip install * langflow-base version * fix: fix retrieverTool component (#4201) ♻️ (RetrieverTool.py): refactor build method signature to accept additional keyword arguments for future extensibility * Fixed save modal not exiting * fix: object has no attribute 'set_event_manager' (#4200) * 🐛 (base.py): fix AttributeError by checking if custom_component has set_event_manager method before calling it * 📝 (base.py): Import Component from langflow.custom to improve code readability and maintainability ♻️ (base.py): Refactor code to use isinstance() method for checking if custom_component is an instance of Component * Refactor: Eliminate Global Variables for Improved Code Maintainability_fix_release (#4208) Refactor: Eliminate Global Variables for Improved Code Maintainability - Replaced global variables with local variables or class attributes. - Enhanced code readability and reduced potential side effects. * fix: Update example (#4204) update example * fix: avoids error NameError: name 'MAX_NUMBER_OF_FIELDS' is not defin… (#4203) fix: avoids error NameError: name 'MAX_NUMBER_OF_FIELDS' is not defined and fixes build method Co-authored-by: Edwin Jose <[email protected]> * fix: unexpected keyword argument 'code' -> SQLExecutor and SQLDatabase (#4230) 🔧 (SQLDatabase.py): update build method signature to accept additional keyword arguments for future extensibility 🔧 (SQLExecutor.py): update method signature to accept additional keyword arguments for future extensibility * lock httptools to 0.6.4 * Move ChatInput import to within flow_component fixture in conftest.py * Simplify error message formatting in test cases for data components * Add readme to dockerfile * build: dockerfile with entrypoint (#4062) Adds a dockerfile with an entrypoint for use with Datastax Langflow * fixes the leading v for checking out commits correctly * fixes on more version checkout for docker build * ✨ (authContext.tsx): Add functionality to fetch global variables on authentication 🔧 (api.tsx): Replace universal-cookie import with react-cookie for consistency 🔧 (authStore.ts): Replace universal-cookie import with react-cookie for consistency 🔧 (use-get-global-variables.ts): Add check to only fetch global variables if user is authenticated ✨ (use-get-mutation-global-variables.ts): Add mutation function to fetch and update global variables 🔧 (authStore.ts): Replace universal-cookie import with react-cookie for consistency * [autofix.ci] apply automated fixes * revert changes to workflows * upgrade lockfile * update pyproject versions * update lockfile again * ⬆️ (pyproject.toml): upgrade langflow-base dependency to version 0.0.99 * ⬆️ (pyproject.toml): downgrade version from 0.0.99 to 0.0.97 to align with project requirements and dependencies. * ⬆️ (pyproject.toml): downgrade langflow-base dependency version from 0.0.99 to 0.0.97 to resolve compatibility issues * ⬆️ (uv.lock): downgrade langchain-core package version from 0.3.15 to 0.3.12 to resolve compatibility issues with dependencies * ⬆️ (pyproject.toml): upgrade langflow-base dependency to version 0.0.99 to utilize the latest features and improvements --------- Co-authored-by: Sebastián Estévez <[email protected]> Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]> Co-authored-by: Lucas Oliveira <[email protected]> Co-authored-by: Edwin Jose <[email protected]> Co-authored-by: anovazzi1 <[email protected]> Co-authored-by: João <[email protected]> Co-authored-by: Jordan Frazier <[email protected]> Co-authored-by: Jordan Frazier <[email protected]> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
diogocabral
pushed a commit
to headlinevc/langflow
that referenced
this pull request
Nov 26, 2024
…flow-ai#4411) * fix: update assistants client import (langflow-ai#4150) * remove unnecessary patch * remove unnecessary patch * compatible release operator * chore: add opensearch-py dependency (langflow-ai#4134) Add opensearch-py dependency to pyproject.toml * patch version * lock * lock some packages to speed up pip install * langflow-base version * fix: fix retrieverTool component (langflow-ai#4201) ♻️ (RetrieverTool.py): refactor build method signature to accept additional keyword arguments for future extensibility * Fixed save modal not exiting * fix: object has no attribute 'set_event_manager' (langflow-ai#4200) * 🐛 (base.py): fix AttributeError by checking if custom_component has set_event_manager method before calling it * 📝 (base.py): Import Component from langflow.custom to improve code readability and maintainability ♻️ (base.py): Refactor code to use isinstance() method for checking if custom_component is an instance of Component * Refactor: Eliminate Global Variables for Improved Code Maintainability_fix_release (langflow-ai#4208) Refactor: Eliminate Global Variables for Improved Code Maintainability - Replaced global variables with local variables or class attributes. - Enhanced code readability and reduced potential side effects. * fix: Update example (langflow-ai#4204) update example * fix: avoids error NameError: name 'MAX_NUMBER_OF_FIELDS' is not defin… (langflow-ai#4203) fix: avoids error NameError: name 'MAX_NUMBER_OF_FIELDS' is not defined and fixes build method Co-authored-by: Edwin Jose <[email protected]> * fix: unexpected keyword argument 'code' -> SQLExecutor and SQLDatabase (langflow-ai#4230) 🔧 (SQLDatabase.py): update build method signature to accept additional keyword arguments for future extensibility 🔧 (SQLExecutor.py): update method signature to accept additional keyword arguments for future extensibility * lock httptools to 0.6.4 * Move ChatInput import to within flow_component fixture in conftest.py * Simplify error message formatting in test cases for data components * Add readme to dockerfile * build: dockerfile with entrypoint (langflow-ai#4062) Adds a dockerfile with an entrypoint for use with Datastax Langflow * fixes the leading v for checking out commits correctly * fixes on more version checkout for docker build * ✨ (authContext.tsx): Add functionality to fetch global variables on authentication 🔧 (api.tsx): Replace universal-cookie import with react-cookie for consistency 🔧 (authStore.ts): Replace universal-cookie import with react-cookie for consistency 🔧 (use-get-global-variables.ts): Add check to only fetch global variables if user is authenticated ✨ (use-get-mutation-global-variables.ts): Add mutation function to fetch and update global variables 🔧 (authStore.ts): Replace universal-cookie import with react-cookie for consistency * [autofix.ci] apply automated fixes * revert changes to workflows * upgrade lockfile * update pyproject versions * update lockfile again * ⬆️ (pyproject.toml): upgrade langflow-base dependency to version 0.0.99 * ⬆️ (pyproject.toml): downgrade version from 0.0.99 to 0.0.97 to align with project requirements and dependencies. * ⬆️ (pyproject.toml): downgrade langflow-base dependency version from 0.0.99 to 0.0.97 to resolve compatibility issues * ⬆️ (uv.lock): downgrade langchain-core package version from 0.3.15 to 0.3.12 to resolve compatibility issues with dependencies * ⬆️ (pyproject.toml): upgrade langflow-base dependency to version 0.0.99 to utilize the latest features and improvements --------- Co-authored-by: Sebastián Estévez <[email protected]> Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]> Co-authored-by: Lucas Oliveira <[email protected]> Co-authored-by: Edwin Jose <[email protected]> Co-authored-by: anovazzi1 <[email protected]> Co-authored-by: João <[email protected]> Co-authored-by: Jordan Frazier <[email protected]> Co-authored-by: Jordan Frazier <[email protected]> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
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 includes significant updates to the GitHub Actions workflows and Docker build configurations. The changes aim to support new build types, improve version handling, and streamline the Docker build process.
Workflow Enhancements:
.github/workflows/docker-build.yml: Added support for building Docker images with a new version format and additional checks for version inputs. Introducedversion-no-voutput to handle versions without a leading 'v'. [1] [2] [3] [4] [5] [6] [7].github/workflows/release.yml: Added a new job to build Docker images for Langflow with an entrypoint (main-ep). [1] [2].github/workflows/release_nightly.yml: Introduced a new job to build nightly Docker images for Langflow with an entrypoint. [1] [2] [3] [4]Dockerfile Updates:
docker/build_and_push.Dockerfile: AddedREADME.mdto the Docker build context to ensure it is included in the build.docker/build_and_push_ep.Dockerfile: Created a new Dockerfile for building Langflow with an entrypoint, including steps for building dependencies and setting up the runtime environment.