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

Warning about benchmarktools #684

Closed
wants to merge 1 commit into from

Conversation

evelyne-ringoot
Copy link

Related to #683, proposing to add a warning regarding the use of benchmarktools

Added warning about benchmarktools
@luraess
Copy link
Collaborator

luraess commented Oct 5, 2024

Thanks for the addition. It is indeed valuable to have some notes about potential limitations using e.g. BenchmarkTools with AMDGPU. One could however relax slightly the formulation as BenchmarkTools may slow down code in some cases but this may not be generalisable to all cases. Also, it would be valuable to have a self contained example and potentially a direct comparison good vs bad perf such that a user could directly run the suggested code snippet. This could and should be further adapted depending on the outcome of investigating #683 .

@vchuravy
Copy link
Member

vchuravy commented Oct 5, 2024

In addition, the same performance issues are observed when using benchmarktools, which relies on copies.

I am surprised by this statement. BenchmarkTools does not introduce copies

@evelyne-ringoot
Copy link
Author

In addition, the same performance issues are observed when using benchmarktools, which relies on copies.

I am surprised by this statement. BenchmarkTools does not introduce copies

My bad, I think I might've put something on benchmarktools that is not actually its fault. I thought I saw weird performance results with benchmarktoold before, but I cannot manage to reproduce them right now so closing this for now

@luraess
Copy link
Collaborator

luraess commented Oct 5, 2024

In some case, in the "past", I also had weird or failing AMDGPU test when using BenchmarkTools which forced me to things in a more manual fashion. I did no longer bump into such failing configs recently though...

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