Skip to content

Conversation

@LAhmos
Copy link
Collaborator

@LAhmos LAhmos commented Jul 4, 2025

I moved microbenchmarks from tuner, to here.

I kept correlation the same as the slightly different. Correlation apps added under ubench dir with *_corr naming.

@LAhmos LAhmos requested review from JRPan and tgrogers July 4, 2025 17:41
Copy link
Contributor

@tgrogers tgrogers left a comment

Choose a reason for hiding this comment

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

I think this is fine, so long as we aren't deleting any uBenchs.

@tgrogers tgrogers merged commit d974177 into accel-sim:dev Jul 4, 2025
1 check passed
@cesar-avalos3
Copy link
Contributor

I think there might be some binaries in this PR

@tgrogers
Copy link
Contributor

tgrogers commented Jul 4, 2025

shoot.
Where?

@LAhmos
Copy link
Collaborator Author

LAhmos commented Jul 4, 2025

Yeah , i will clean that

@tgrogers
Copy link
Contributor

tgrogers commented Jul 4, 2025

oh crap - yeah I missed that. Please try to be more careful about this.
You will need to obliterate them from history, or the repo will be huge. A simple delete won't do it.

LAhmos pushed a commit that referenced this pull request Jul 5, 2025
merge microbenchmarks with tuner ubench
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