Skip to content

fix: 111 #1581

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

Closed
wants to merge 1 commit into from
Closed

fix: 111 #1581

wants to merge 1 commit into from

Conversation

shaohuzhang1
Copy link
Contributor

fix: 111

Copy link

f2c-ci-robot bot commented Nov 8, 2024

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Copy link

f2c-ci-robot bot commented Nov 8, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wxg0103 wxg0103 closed this Nov 8, 2024
@@ -305,6 +305,7 @@ const openAIParamSettingDialog = (modelId: string) => {
if (modelId) {
AIModeParamSettingDialogRef.value?.open(modelId, id, chat_data.value.model_params_setting)
}
console.log(chat_data.value.model_params_setting)
}

function refreshParam(data: any) {

Choose a reason for hiding this comment

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

代码在变量命名和空语句方面可能存在一些混淆之处。以下是一些建议:

  1. console.log 之前添加一个闭合括号以避免拼写错误。
  2. 尽可能地使用函数来组织相关的代码块,并且尽量简化这些功能之间的关系。

请注意,这仅仅是一种格式化的建议,并不一定适用于所有情况,请根据具体的上下文进行调整。

@maninhill maninhill deleted the pr@main@fix_111 branch November 13, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants