Skip to content
This repository was archived by the owner on Mar 25, 2026. It is now read-only.

Fix to run benchmark on remote server#248

Merged
KotaTakahashi9320 merged 5 commits intomainfrom
feature/takahashi/fix_benchmark
Jun 30, 2023
Merged

Fix to run benchmark on remote server#248
KotaTakahashi9320 merged 5 commits intomainfrom
feature/takahashi/fix_benchmark

Conversation

@KotaTakahashi9320
Copy link
Copy Markdown
Collaborator

@KotaTakahashi9320 KotaTakahashi9320 commented Jun 30, 2023

Summary

Fix to run benchmark on remote server

Purpose

Because github action has a 6 hour limit and the benchmark stops in the middle of the action.

Contents

Fix github action to send only requests from github action
Add shell script to run benchmark
Sent result to Slack

Testing Methods Performed

Example of a message sent to slack
スクリーンショット 2023-06-30 17 28 34

Comment thread benchmark_src/benchmark_runner.sh
Comment thread benchmark_src/benchmark_runner.sh
Copy link
Copy Markdown
Contributor

@come2ry come2ry left a comment

Choose a reason for hiding this comment

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

AC

@KotaTakahashi9320 KotaTakahashi9320 changed the title Feature/takahashi/fix benchmark Fix benchmark to run on a stepping stone server Jun 30, 2023
@KotaTakahashi9320 KotaTakahashi9320 marked this pull request as ready for review June 30, 2023 08:30
@KotaTakahashi9320 KotaTakahashi9320 changed the title Fix benchmark to run on a stepping stone server Fix to run benchmark on remote server Jun 30, 2023
@KotaTakahashi9320 KotaTakahashi9320 force-pushed the feature/takahashi/fix_benchmark branch from 83ea54f to 90b9e52 Compare June 30, 2023 08:37
@KotaTakahashi9320 KotaTakahashi9320 merged commit 18d5fd1 into main Jun 30, 2023
@KotaTakahashi9320 KotaTakahashi9320 deleted the feature/takahashi/fix_benchmark branch June 30, 2023 09:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants