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

Update tornado-benchmarks.py, add --properties option #450

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

yc128
Copy link
Contributor

@yc128 yc128 commented Jun 15, 2024

Description

Add --properties option to display all dimensions and sizes for all benchmarks

Backend/s tested

Mark the backends affected by this PR.

  • OpenCL
  • PTX
  • SPIRV

OS tested

Mark the OS where this PR is tested.

  • Linux
  • OSx
  • Windows

Did you check on FPGAs?

If it is applicable, check your changes on FPGAs.

  • Yes
  • No

How to test the new patch?

Run the benchmark script using --properties option
python %TORNADO_SDK%\bin\tornado-benchmarks.py --properties


Add --properties option to display all dimensions and sizes for all benchmarks
@CLAassistant
Copy link

CLAassistant commented Jun 15, 2024

CLA assistant check
All committers have signed the CLA.

@stratika
Copy link
Collaborator

hi @yc128, please sign the CLA and we will continue with the code review process. To add some context, this PR adds a small feature that displays the dimensions and sizes supported by the tornado-benchmarks.py script. This is the output I get:

tornado-benchmarks.py --properties
montecarlo, dims=1, size=512
montecarlo, dims=1, size=1024
montecarlo, dims=1, size=2048
montecarlo, dims=1, size=4096
montecarlo, dims=1, size=8192
montecarlo, dims=1, size=16384
montecarlo, dims=1, size=32798
montecarlo, dims=1, size=65536
montecarlo, dims=1, size=1048576
nbody, dims=1, size=512
nbody, dims=1, size=1024
nbody, dims=1, size=2048
nbody, dims=1, size=4096
nbody, dims=1, size=16384
nbody, dims=1, size=327684
saxpy, dims=1, size=512
saxpy, dims=1, size=1024
saxpy, dims=1, size=2048
saxpy, dims=1, size=4096
saxpy, dims=1, size=8192
saxpy, dims=1, size=16384
saxpy, dims=1, size=32798
saxpy, dims=1, size=65536
saxpy, dims=1, size=131072
saxpy, dims=1, size=262144
saxpy, dims=1, size=524288
saxpy, dims=1, size=1048576
saxpy, dims=1, size=2097152
saxpy, dims=1, size=4194304
sgemm, dims=2, size=128
sgemm, dims=2, size=256
sgemm, dims=2, size=512
sgemm, dims=2, size=1024
sgemm, dims=2, size=2048
blackscholes, dims=1, size=512
blackscholes, dims=1, size=1024
blackscholes, dims=1, size=2048
blackscholes, dims=1, size=4096
blackscholes, dims=1, size=8192
blackscholes, dims=1, size=16384
blackscholes, dims=1, size=32798
blackscholes, dims=1, size=65536
blackscholes, dims=1, size=1048576
blackscholes, dims=1, size=4194304
dft, dims=1, size=256
dft, dims=1, size=512
dft, dims=1, size=1024
dft, dims=1, size=2048
dft, dims=1, size=4096
dft, dims=1, size=8192
blurFilter, dims=2, size=256
blurFilter, dims=2, size=512
blurFilter, dims=2, size=1024
blurFilter, dims=2, size=2048
blurFilter, dims=2, size=8192
blurFilter, dims=2, size=16384
juliaset, dims=2, size=512
juliaset, dims=2, size=1024
juliaset, dims=2, size=2048
juliaset, dims=2, size=4096
juliaset, dims=2, size=8192

@jjfumero
Copy link
Member

Is this for the data base applications for performance testing?

@stratika
Copy link
Collaborator

Is this for the data base applications for performance testing?

Yes, this particular PR does not do anything regarding testing. But it is to populate metadata regarding (benchmarks, dimensions, sizes) in the database.

@jjfumero jjfumero added the enhancement New feature or request label Jun 18, 2024
Copy link
Collaborator

@stratika stratika left a comment

Choose a reason for hiding this comment

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

LGTM

@jjfumero jjfumero merged commit e843843 into beehive-lab:develop Jun 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

4 participants