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

Add benchmark inference_tract #36

Closed

Conversation

jlb6740
Copy link
Collaborator

@jlb6740 jlb6740 commented May 21, 2024

No description provided.

@jlb6740 jlb6740 requested a review from rahulchaphalkar May 21, 2024 06:23
@rahulchaphalkar
Copy link
Collaborator

Need to update config.inc with new sha, else shows run invalid.

@@ -29,6 +29,9 @@ echo "Revision:" $REVISION
echo "Build Sha:" $BUILD_SHA "vs" $CURRENT_BUILD_SHA "(calculated)"
echo ""

# Run pre-docker build setup scripts
(cd ./benchmarks/inference_tract/rust-benchmark && ./setup.sh)
Copy link
Collaborator

Choose a reason for hiding this comment

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

setup.sh doesn't exist at that directory. Is the benchmark silently failing (since it didn't download a model)?

@jlb6740 jlb6740 force-pushed the add-inference-tract-workload branch from 194916f to 3b55fcb Compare May 23, 2024 21:07
@rahulchaphalkar
Copy link
Collaborator

@jlb6740 I see some commits, is this ready for review?

@jlb6740
Copy link
Collaborator Author

jlb6740 commented Jun 12, 2024

Dropping this PR in favor of: #37. Will address SHA comment there. The config question should already be addressed there .. I use a build.rs script instead.

@jlb6740 jlb6740 closed this Jun 12, 2024
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