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

update: Add support for post tools, add round limits, per-round token… #1230

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

xingyunyang01
Copy link
Collaborator

… configuration and per-round request time configuration #1206

Ⅰ. Describe what this PR did

fix #1206
增加了post method
增加了轮次限制,单论token限制以及单论请求时间限制
redis的支持,我没有加,因为考虑到有轮次的限制,不会出现内存装不下历史会话的情况,且由于每次请求完成后,历史记录就没用了,需要清掉,因此存储到持久化存储中没意义。

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

… configuration and per-round request time configuration #1206
@codecov-commenter
Copy link

codecov-commenter commented Aug 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 35.98%. Comparing base (ef31e09) to head (fea4e32).
Report is 51 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1230      +/-   ##
==========================================
+ Coverage   35.91%   35.98%   +0.07%     
==========================================
  Files          69       69              
  Lines       11576     9500    -2076     
==========================================
- Hits         4157     3419     -738     
+ Misses       7104     5766    -1338     
  Partials      315      315              

see 67 files with indirect coverage changes

@johnlanni
Copy link
Collaborator

cc @CH3CHO

Copy link
Collaborator

@CH3CHO CH3CHO left a comment

Choose a reason for hiding this comment

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

config.go 里有一个 apiStrcut 的拼写错误

@CH3CHO CH3CHO merged commit 895f17f into alibaba:main Aug 22, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ai-agent插件后续开发计划
4 participants