Skip to content

Conversation

@hvagadia
Copy link
Contributor

Checkout README.md for instructions.

@hvagadia hvagadia requested a review from a team as a code owner December 16, 2025 18:17
@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

Copy link
Contributor

@nvzhihanj nvzhihanj left a comment

Choose a reason for hiding this comment

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

Thank you Harshil


- Model: [Wan-AI/Wan2.2-T2V-A14B-Diffusers](https://huggingface.co/Wan-AI/Wan2.2-T2V-A14B-Diffusers)
- VBench: [GitHub](https://github.com/Vchitect/VBench)
- MLPerf: [Inference](https://github.com/mlcommons/inference)
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing:

  1. Reference accuracy score section for VBench at original accuracy (BF16)
  2. Supported hardware (I suppose this is Hopper and pre-hopper based on the torch/cuda version)?
  3. mlperf.conf change for performance sample count and related settings
  4. submission checker changes

Copy link
Contributor

Choose a reason for hiding this comment

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

Submission checker can go into a separate PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

1, 2 done. 3 and 4 can likely be clubbed with Loadgen changes Pablo is helping us with.

@@ -0,0 +1,46 @@
FROM pytorch/pytorch:2.5.1-cuda12.1-cudnn9-devel
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like this is pre-Blackwell, recommend adding the supported hardware in the README

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noted in readme.


try:
print("Starting download...")
snapshot_download(
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we ping to a certain commit of the model? (or at least note it in the readme)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noted commit id in the Readme. There isn't much activity in huggingface model repo, so we should be fine.

@@ -0,0 +1,39 @@
# Text-to-Video Benchmark
Copy link
Contributor

Choose a reason for hiding this comment

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

Regarding the folder name, it should be text_to_video/Wan2.2-T2V-A14B so we can add new models in the future. (We didn't catch it in the text_to_image)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved to model specific folder.

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.

3 participants