Skip to content
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

Superbenchmark executor: online inference config file and custom repo link #369

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

LiYueqian-James
Copy link
Contributor

Added support for

  1. Feeding an online inference config file as a link, without the need to zip into packages and upload to storage account
  2. Feeding a different superbenchmark repo link other the default microsoft SB repo. Some users have added more benchmark to their own version of superbench and can use this method run them in VC.

/// <summary>
/// Link to the superbench repo.
/// </summary>
public string RepositoryLink
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to change the repository link? Even if we need to support extra config?

string configFileName = Path.GetFileName(configFileUri.AbsolutePath);
string configFullPath = this.PlatformSpecifics.Combine(this.SuperBenchmarkDirectory, configFileName);

using (var client = new HttpClient())
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's not add un-unit testable code here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Are cmd line based methods preferred? like wget or curl?

@yangpanMS
Copy link
Contributor

Could we do this instead: you are allowed to use extra dependency (for example git clone) in the profile, and you give it a package name. You then could reference the file you cloned by path.

@LiYueqian-James
Copy link
Contributor Author

Could we do this instead: you are allowed to use extra dependency (for example git clone) in the profile, and you give it a package name. You then could reference the file you cloned by path.

so it's not just about inference config file. some people have created their own SB benchmarks (in their own repo) and want to run those. The official one do not have these benchmarks. That's why I added a parameter for repoLink

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants