Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

fix: Remove 'refresh_button' from DropdownInput in AgentComponent configuration#4471

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
italojohnny merged 3 commits intomainfromfix-remove-agent-button
Nov 8, 2024

Conversation

ogabrielluiz
Copy link
Contributor

This PR removes the 'refresh_button' option from the DropdownInput in the AgentComponent configuration, streamlining the component's interface and functionality.

@dosubotdosubotbot added the size:XSThis PR changes 0-9 lines, ignoring generated files. labelNov 8, 2024
@github-actionsgithub-actionsbot added the bugSomething isn't working labelNov 8, 2024
@codspeed-hqCodSpeed HQ
Copy link

codspeed-hqbot commentedNov 8, 2024•
edited
Loading

CodSpeed Performance Report

Merging#4471 willdegrade performances by 13.34%

Comparingfix-remove-agent-button (30122ad) withmain (a4863bb)

Summary

⚡ 4 improvements
❌ 1 regressions
✅ 10 untouched benchmarks

⚠️Please fix the performance issues oracknowledge them on CodSpeed.

Benchmarks breakdown

Benchmarkmainfix-remove-agent-buttonChange
⚡test_build_flow5,519.8 ms690 ms×8
⚡test_invalid_run_with_input_type_chat20.6 ms17.6 ms+17.26%
⚡test_successful_run_with_input_type_any449.6 ms397.5 ms+13.13%
❌test_successful_run_with_input_type_text419.6 ms484.1 ms-13.34%
⚡test_successful_run_with_output_type_any528.3 ms449.5 ms+17.54%

@dosubotdosubotbot added size:LThis PR changes 100-499 lines, ignoring generated files. and removed size:XSThis PR changes 0-9 lines, ignoring generated files. labelsNov 8, 2024
@dosubotdosubotbot added size:XSThis PR changes 0-9 lines, ignoring generated files. and removed size:LThis PR changes 100-499 lines, ignoring generated files. labelsNov 8, 2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM

@dosubotdosubotbot added the lgtmThis PR has been approved by a maintainer labelNov 8, 2024
@italojohnnyitalojohnnyenabled auto-merge (squash)November 8, 2024 17:27
@italojohnnyitalojohnny merged commitc5495c4 intomainNov 8, 2024
28 checks passed
@italojohnnyitalojohnny deleted the fix-remove-agent-button branchNovember 8, 2024 17:43
diogocabral pushed a commit to headlinevc/langflow that referenced this pull requestNov 26, 2024
…figuration (langflow-ai#4471)* Remove 'refresh_button' from DropdownInput in AgentComponent configuration* Add 'Current Date' tool option to AgentComponent inputs and update response method* Disable refresh button in Agent Flow starter project configuration
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@edwinjosechittilappillyedwinjosechittilappillyedwinjosechittilappilly approved these changes

Assignees

No one assigned

Labels

bugSomething isn't workinglgtmThis PR has been approved by a maintainersize:XSThis PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ogabrielluiz@edwinjosechittilappilly@italojohnny
[8]ページ先頭
©2009-2025 Movatter.jp