Skip to content

feat(sys-advisor): get borwein model input and store it in cache#1006

Merged
luomingmeng merged 1 commit intokubewharf:mainfrom
syc4704413:dev/syc/borwein-model-input
Nov 13, 2025
Merged

feat(sys-advisor): get borwein model input and store it in cache#1006
luomingmeng merged 1 commit intokubewharf:mainfrom
syc4704413:dev/syc/borwein-model-input

Conversation

@syc4704413
Copy link
Copy Markdown
Collaborator

@syc4704413 syc4704413 commented Oct 24, 2025

What type of PR is this?

Features

@syc4704413 syc4704413 added the workflow/need-review review: test succeeded, need to review label Oct 24, 2025
@syc4704413 syc4704413 force-pushed the dev/syc/borwein-model-input branch from ea84713 to 87949c1 Compare October 24, 2025 10:02
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 24, 2025

Codecov Report

❌ Patch coverage is 64.03509% with 123 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.87%. Comparing base (499f48f) to head (8e8bc8c).
⚠️ Report is 64 commits behind head on main.

Files with missing lines Patch % Lines
...gin/inference/modelinputfetcher/generic/generic.go 31.81% 45 Missing ⚠️
.../agent/metric/provisioner/malachite/provisioner.go 62.00% 38 Missing ⚠️
...in/inference/modelresultfetcher/borwein/borwein.go 52.38% 14 Missing and 6 partials ⚠️
pkg/agent/sysadvisor/plugin/inference/inference.go 72.41% 5 Missing and 3 partials ⚠️
...ric/provisioner/malachite/provisioner_calculate.go 91.04% 5 Missing and 1 partial ⚠️
pkg/agent/sysadvisor/metacache/metacache.go 71.42% 2 Missing and 2 partials ⚠️
pkg/metaserver/agent/pod/pod.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1006      +/-   ##
==========================================
+ Coverage   59.65%   59.87%   +0.21%     
==========================================
  Files         682      689       +7     
  Lines       63929    65114    +1185     
==========================================
+ Hits        38137    38985     +848     
- Misses      21385    21672     +287     
- Partials     4407     4457      +50     
Flag Coverage Δ
unittest 59.87% <64.03%> (+0.21%) ⬆️

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.

@syc4704413 syc4704413 force-pushed the dev/syc/borwein-model-input branch 8 times, most recently from 209c9c5 to d87395d Compare October 30, 2025 06:31
@syc4704413 syc4704413 force-pushed the dev/syc/borwein-model-input branch from d87395d to 95860c0 Compare November 7, 2025 03:32
Comment thread pkg/metaserver/agent/pod/pod.go
Comment thread pkg/agent/sysadvisor/plugin/inference/modelinputfetcher/borwein/borwein.go Outdated
@syc4704413 syc4704413 force-pushed the dev/syc/borwein-model-input branch 7 times, most recently from ef5afe6 to cbcc7a7 Compare November 12, 2025 08:13
@syc4704413 syc4704413 force-pushed the dev/syc/borwein-model-input branch from cbcc7a7 to 8e8bc8c Compare November 12, 2025 08:20
@luomingmeng luomingmeng merged commit 2975586 into kubewharf:main Nov 13, 2025
23 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow/need-review review: test succeeded, need to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants