Skip to content

Conversation

@DaltheCow
Copy link
Collaborator

Summary

The run info will now grab the dataset and if the model is pulled from HF it will grab the model size

Details

  • Dataset added to run info
  • Model size added to run info

Test Plan

  • Function which grabs dataset is tested

github-actions bot pushed a commit that referenced this pull request Oct 7, 2025
@github-actions
Copy link

github-actions bot commented Oct 7, 2025

🎉 Live Preview: Click here to view the live version
Last updated: e88579f

@DaltheCow DaltheCow added the UI Front-end workstream label Oct 7, 2025
@DaltheCow DaltheCow changed the title Add dataset and model size to the UI Features/add dataset and model size to the UI Oct 7, 2025
Copy link
Collaborator

@sjmonson sjmonson left a comment

Choose a reason for hiding this comment

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

One question, just resolve it and I'll approve

bm.run_stats.start_time for bm in benchmarks if bm.start_time is not None
)
response = httpx.get(f"https://huggingface.co/api/models/{model}")
model_json = response.json()
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this safe from an error response?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm splitting time between this team and another team now but I will get to this. Definitely need to handle an error response

sjmonson added a commit that referenced this pull request Oct 10, 2025
feature/ui/add-dataset-and-model-size
Base automatically changed from features/refactor/base to main October 17, 2025 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI Front-end workstream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants