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

Restructure to match more common ruby conventions #280

Merged
merged 1 commit into from
Jun 26, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ site (hosted at speed.yjit.org) with a simple command:
This will move files from `./data/` into a `build` directory,
generate all the html files, and start a web server where you can view the site at `localhost:8000`.

Some of the reports are built using `lib/yjit-metrics` and have templates in
`lib/yjit-metrics/report_templates`.
Some of the reports are built using `lib/yjit_metrics` and have templates in
`lib/yjit_metrics/report_templates`.
The rest of the files that build the site are found beneath `site`.
There are `erb` files to generate additional pages
and the script that does all the file rendering in `site/_framework/`.
Expand Down Expand Up @@ -87,7 +87,7 @@ Then a quick "git diff" in the pages directory can show you what, if anything, c
### Output Format

Changes to the JSON format may require a bump to the `version` entry (`lib/yjit-metrics.rb`)
and translation logic in the code that processes the data (`lib/yjit-metrics/bench-results.rb`).
and translation logic in the code that processes the data (`lib/yjit_metrics/result_set.rb`).

### Tests

Expand Down
2 changes: 1 addition & 1 deletion basic_benchmark.rb
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
require "optparse"
require "fileutils"
require "etc"
require_relative "lib/yjit-metrics"
require_relative "lib/yjit_metrics"

# Default settings for benchmark sampling
DEFAULT_WARMUP_ITRS = 15 # Number of un-reported warmup iterations to run before "counting" benchmark runs
Expand Down
2 changes: 1 addition & 1 deletion basic_report.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

require "json"
require "optparse"
require_relative "lib/yjit-metrics"
require_relative "lib/yjit_metrics"

RESULT_SET = YJITMetrics::ResultSet.new

Expand Down
2 changes: 1 addition & 1 deletion continuous_reporting/benchmark_and_update.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env ruby

require_relative "../lib/yjit-metrics"
require_relative "../lib/yjit_metrics"

require 'fileutils'
require 'net/http'
Expand Down
2 changes: 1 addition & 1 deletion continuous_reporting/create_json_params_file.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
require "optparse"
require "json"

require_relative "../lib/yjit-metrics"
require_relative "../lib/yjit_metrics"

# A particular run through the benchmarking system has a number of important parameters.
# Most, though not all, are captured in the JSON file produced here. In general,
Expand Down
2 changes: 1 addition & 1 deletion continuous_reporting/file_benchmark_data_into_raw.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
require "fileutils"
require "optparse"

require_relative "../lib/yjit-metrics"
require_relative "../lib/yjit_metrics"

YJIT_RAW_DATA_REPO = File.join(YJITMetrics::ContinuousReporting::RAW_BENCHMARK_ROOT, "raw_benchmark_data")

Expand Down
2 changes: 1 addition & 1 deletion continuous_reporting/generate_and_upload_reports.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
require "optparse"
require "rbconfig"

require_relative "../lib/yjit-metrics"
require_relative "../lib/yjit_metrics"

### Required repos, etc, to build from

Expand Down
Loading
Loading