Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Stacked] Migrate getCumulativeFeedbackTimeseries#5269

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
shuyangli wants to merge3 commits intomain
base:main
Choose a base branch
Loading
fromsl/migrate-feedback-queries-to-gateway

Conversation

@shuyangli
Copy link
Member

@shuyanglishuyangli commentedDec 18, 2025
edited
Loading

  • Add new route handlers for /internal/feedback/timeseries

Also delete getFeedbackByVariant since it's unused.

@shuyanglishuyangli changed the title[Stacked] Migrate getFeedbackByVariant and getCumulativeFeedbackTimeseries[Stacked] Migrate getCumulativeFeedbackTimeseriesDec 18, 2025
@shuyanglishuyangli mentioned this pull requestDec 18, 2025
53 tasks
@shuyanglishuyangliforce-pushed thesl/migrate-feedback-queries-to-gateway branch from733c66f to3ed370dCompareDecember 18, 2025 16:44
@shuyanglishuyangliforce-pushed thesl/migrate-get-function-throughput-by-variant branch from4a3c442 to2f91657CompareDecember 18, 2025 16:57
@shuyanglishuyangliforce-pushed thesl/migrate-feedback-queries-to-gateway branch 2 times, most recently fromd01f70b to96bb612CompareDecember 18, 2025 17:08
@shuyanglishuyangliforce-pushed thesl/migrate-get-function-throughput-by-variant branch from2f91657 toc604a6dCompareDecember 18, 2025 18:38
…window filteringRefactors the time-based filtering in ClickHouse inference stats queries:- Changed time_window_to_ms to time_window_to_duration returning std::time::Duration- Use ClickHouse's UUIDv7ToDateTime function for timestamp comparison instead of  manual bit manipulation of UUIDv7 format- Pass time delta in seconds to ClickHouse, use INTERVAL SECOND for arithmetic- Preserves original semantics of filtering relative to max timestamp in the dataFix inference stats e2e test assertions- Check RFC 3339 format with milliseconds on raw HTTP response body instead of  re-serializing DateTime (chrono's default serializer omits .000 for zero ms)- Relax variant count assertion from 2+ to 1+ for fixture data flexibility
@shuyanglishuyangliforce-pushed thesl/migrate-get-function-throughput-by-variant branch fromc604a6d tocc51a91CompareDecember 18, 2025 18:45
- Add new route handler for /internal/feedback/timeseries- Add unit tests using mock ClickHouse- Add e2e tests for the new endpoint- Add e2e ClickHouse tests for getCumulativeFeedbackTimeseries- Add getCumulativeFeedbackTimeseries method to TensorZeroClient- Update function route to use TensorZeroClient instead of DatabaseClient for cumulative feedback- Remove getCumulativeFeedbackTimeseries from tensorzero-node NAPI bindings
@shuyanglishuyangliforce-pushed thesl/migrate-feedback-queries-to-gateway branch from96bb612 toe7b18feCompareDecember 18, 2025 19:15
@shuyanglishuyangliforce-pushed thesl/migrate-get-function-throughput-by-variant branch from341c0a8 toc329576CompareDecember 18, 2025 21:55
@shuyanglishuyangliforce-pushed thesl/migrate-get-function-throughput-by-variant branch 2 times, most recently from32e9aee toe538951CompareDecember 18, 2025 22:41
Base automatically changed fromsl/migrate-get-function-throughput-by-variant tomainDecember 19, 2025 01:03
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shuyangli

[8]ページ先頭

©2009-2025 Movatter.jp