Skip to content

Conversation

yangw-dev
Copy link
Contributor

@yangw-dev yangw-dev commented Oct 15, 2025

Overview

  1. add dynamic query builder in HUD
  2. add default and microOperatar fetcher in both listMetadata and get_time_series
  3. make the compiler benchmark query depends on workflows instead of commits
    • when fetch based on commits, sometimes additinonal worklfows pop up

BE

  1. Move FE and Backend code into two folders for benchmark
    FE and Backend should not depends/share same module, make it clear the use cases

Copy link

vercel bot commented Oct 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
torchci Ready Ready Preview Oct 16, 2025 6:58pm

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 15, 2025
@yangw-dev yangw-dev requested a review from huydhn October 15, 2025 00:13
@yangw-dev yangw-dev changed the title Dynamic QueryBuilder For benchmark Dynamic QueryBuilder For benchmark + ListMetadataApi Oct 15, 2025
@yangw-dev yangw-dev changed the title Dynamic QueryBuilder For benchmark + ListMetadataApi Dynamic QueryBuilder For benchmark + ListMetadataApi & get_time_series Api Oct 15, 2025
@yangw-dev
Copy link
Contributor Author

sorry it seems the file moving messed up in this pr, and make things looks like new file. @huydhn
for main review, I@ you

@@ -0,0 +1,185 @@
import { queryClickhouse } from "lib/clickhouse";

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@huydhn
this is the key of dynamica builder for sql schema

@@ -0,0 +1,181 @@
import {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@huydhn
the main endpoint for listMetadata builder

@@ -0,0 +1,42 @@
import {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

main module where we store all fetchers @huydhn

};
}

export class PytorchOperatorMicrobenchmarkMetadataFetcher
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@huydhn

customized list metadata option api for micro torch operator api

* - applyFormat(): to format the data to the desired format, default is time series
*
*/
export class BenchmarkDataQuery extends ExecutableQueryBase {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@huydhn getBenchmarkDataQuery
this is the main backend fetch to fetch benchmark data

* It inherits method from BenchmarkDataQuery
*
*/
export class PytorchOperatorMicroBenchmarkDataFetcher
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@huydhn this is the customized fetcher for PytorchOperatorMicroBenchmarkDataFetcher

it has customized extra_keys
and additioanl filter logics to fetch data only has operatorName

@yangw-dev yangw-dev merged commit 6e3763d into main Oct 16, 2025
5 checks passed
@yangw-dev yangw-dev deleted the torchao branch October 16, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants