Skip to content

fix: remove sumi- prefix to prevent model misspell#4459

Merged
ensorrow merged 1 commit intov3.8from
fix/sumi-builtin-name
Mar 10, 2025
Merged

fix: remove sumi- prefix to prevent model misspell#4459
ensorrow merged 1 commit intov3.8from
fix/sumi-builtin-name

Conversation

@ensorrow
Copy link
Copy Markdown
Contributor

Types

  • 🐛 Bug Fixes

Background or solution

fix: remove sumi- prefix to prevent model misspell

Changelog

fix: remove sumi- prefix to prevent model misspell

@ensorrow ensorrow requested a review from life2015 March 10, 2025 08:00
@opensumi opensumi Bot added the 🐞 bug Something isn't working label Mar 10, 2025
@ensorrow
Copy link
Copy Markdown
Contributor Author

sumi 不是一个标准的单词,实测下来加上以后,模型容易输出错误(比如拼成 sumbi 之类的)

@ensorrow ensorrow requested review from Aaaaash and erha19 March 10, 2025 08:08
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.27%. Comparing base (907611a) to head (a48497e).
Report is 1 commits behind head on v3.8.

Additional details and impacted files
@@            Coverage Diff             @@
##             v3.8    #4459      +/-   ##
==========================================
- Coverage   53.27%   53.27%   -0.01%     
==========================================
  Files        1661     1661              
  Lines      102534   102534              
  Branches    22179    22187       +8     
==========================================
- Hits        54627    54623       -4     
- Misses      39859    39862       +3     
- Partials     8048     8049       +1     
Flag Coverage Δ
jsdom 48.76% <100.00%> (-0.01%) ⬇️
node 12.20% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ensorrow ensorrow merged commit c6d8019 into v3.8 Mar 10, 2025
@ensorrow ensorrow deleted the fix/sumi-builtin-name branch March 10, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants