-
Notifications
You must be signed in to change notification settings - Fork 251
feat: 支持供应商gpustack与xinference #353
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
base: main
Are you sure you want to change the base?
Conversation
|
@MonkeyCode-AI review |
|
@MonkeyCode-AI 帮我review一下这个代码 |
|
@MonkeyCode-AI 帮我review一下这个代码 |
|
@MonkeyCode-AI review |
|
⏳ MonkeyCode-AI 正在分析,请稍等片刻... |
1 similar comment
|
⏳ MonkeyCode-AI 正在分析,请稍等片刻... |
|
@MonkeyCode-AI 安装离线版所有模型需要多少容量的存储 |
|
@MonkeyCode-AI m3-6Y30处理器是否可以用于安装离线版 |
|
@MonkeyCode-AI review |
1 similar comment
|
@MonkeyCode-AI review |
💬 代码审查评论: 整体来看该 PR 以依赖升级与依赖清理为主,方向合理,但存在较高的构建可复现性与回归风险,需要在合并前重点把关。当前需要明确确认 go.mod/go.sum 与依赖图一致、被移除依赖对应功能未受影响,并补齐后端与 UI 升级后的回归验证。另需尽快处理后端新增封装带来的可维护性问题(变量遮蔽)及潜在 nil 初始化风险以保证稳定性。 📊 统计
🟡 警告 (9)logrus 被标记为 indirect 可能与实际使用不一致
|
|
@MonkeyCode-AI 这个pr主要是什么更新 |
|
MonkeyCode-AI 正在分析任务... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
代码审查结果
将 modelkit 由包级调用改为注入到 ModelHandler 实例中方向正确,但当前实现存在 Go 命名遮蔽导致无法编译的问题,需修复后才能合入。
✨ 代码亮点
- 将对 ModelKit 的调用改为通过 handler 持有实例(h.modelkit),有利于后续测试注入与解耦,且避免了包级全局状态的扩散。
| 🚨 Critical | 💡 Suggestion | |
|---|---|---|
| 1 | 0 | 0 |
| modelkit := modelkit.NewModelKit(logger) | ||
| m := &ModelHandler{usecase: usecase, logger: logger.With("handler", "model"), modelkit: modelkit} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Caution
🚨 局部变量名与导入包名冲突(命名遮蔽),导致无法调用 modelkit.NewModelKit 编译失败
文件中已导入 modelkit "github.com/chaitin/ModelKit/v2/usecase",但在 NewModelHandler 内部使用了 modelkit := modelkit.NewModelKit(logger)。
在 Go 中,短变量声明左侧的 modelkit 会在该作用域内遮蔽同名导入包标识符;同时右侧表达式会在该新作用域中解析,从而使 modelkit.NewModelKit 试图在“局部变量 modelkit”上取字段/方法而不是在包上取函数,导致编译报错(典型报错形态:modelkit.NewModelKit undefined (type *modelkit.ModelKit has no field or method NewModelKit) 或 cannot refer to modelkit.NewModelKit)。
这是确定性的编译级问题,必须修复。
建议: 避免与导入包同名的局部变量;可改为 mk / modelKit 等,并在结构体初始化中对应替换。
modelKit := modelkit.NewModelKit(logger)
m := &ModelHandler{usecase: usecase, logger: logger.With("handler", "model"), modelkit: modelKit}
支持供应商gpustack与xinference
变更类型