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

fixes issue 876 bad assumption about temp path #877

Merged
merged 1 commit into from
Dec 11, 2020
Merged

fixes issue 876 bad assumption about temp path #877

merged 1 commit into from
Dec 11, 2020

Conversation

fbbradheintz
Copy link
Contributor

Description

Fixes #876 - bad assumption about tmp path on Mac

Type of change

Please delete options that are not relevant.

  • [√] Bug fix (non-breaking change which fixes an issue)

Feature/Issue validation/testing

Issue was 100% failure; fixed path assumption, now runs correctly on Mac. Nothing changed for other envs.

  • Logs
Generating Reports...

Writing extracted PredictionTime metrics to /var/folders/dz/74wm7dyd7_x4sc9q2fzm_h8dqwwlkh/T/benchmark/predict.txt

Writing extracted HandlerTime metrics to /var/folders/dz/74wm7dyd7_x4sc9q2fzm_h8dqwwlkh/T/benchmark/handler_time.txt

Writing extracted QueueTime metrics to /var/folders/dz/74wm7dyd7_x4sc9q2fzm_h8dqwwlkh/T/benchmark/waiting_time.txt

Writing extracted WorkerThreadTime metrics to /var/folders/dz/74wm7dyd7_x4sc9q2fzm_h8dqwwlkh/T/benchmark/worker_thread.txt
*Generating CSV output...
*Preparing graphs...
*Preparing Profile graphs...
Working with sampling rate of 10

Test suite execution complete.

Checklist:

  • [n/a] Have you added tests that prove your fix is effective or that this feature works?
  • [n/a] New and existing unit tests pass locally with these changes?
  • [n/a] Has code been commented, particularly in hard-to-understand areas?
  • [n/a] Have you made corresponding changes to the documentation?

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: ae71749
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-win
  • Commit ID: ae71749
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: ae71749
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@maaquib maaquib merged commit b5e0e54 into pytorch:master Dec 11, 2020
@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-cpu
  • Commit ID: ae71749
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-win
  • Commit ID: ae71749
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-neo-ci-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: torch-serve-build-gpu
  • Commit ID: ae71749
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

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.

Apache Benchmark failure on Mac
4 participants