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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove @on_demand_benchmark from memory benchmarks #5950

Closed
trexfeathers opened this issue May 13, 2024 · 0 comments 路 Fixed by #5959
Closed

Remove @on_demand_benchmark from memory benchmarks #5950

trexfeathers opened this issue May 13, 2024 · 0 comments 路 Fixed by #5959
Assignees
@pp-mo
Labels
Type: Performance Type: Testing

Comments

@trexfeathers
Copy link
Contributor

馃摪 Custom Issue

We turned these off in #something

Should be OK to bring these back once #5948 is merged, because tracemalloc appears to be more stable. We can always undo if not.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pp-mo pp-mo

Labels
Type: Performance Type: Testing
Projects
馃毚 Peloton
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove "on-demand" from some benchmarks pp-mo/iris
3 participants
@pp-mo @trexfeathers @HGWright