Skip to content

Remove unnecessary plugins#354

Merged
dhosterman merged 1 commit intomainfrom
remove-unnecessary-plugins
Apr 30, 2024
Merged

Remove unnecessary plugins#354
dhosterman merged 1 commit intomainfrom
remove-unnecessary-plugins

Conversation

@dhosterman
Copy link
Copy Markdown
Collaborator

This dramatically reduces the amount of time it takes to run modelbench --help. It's about 2x as fast on the first run (from 5.5ish seconds to 2.5ish seconds) and about 3x as fast on subsequent runs (from 1ish second to .33ish seconds).

I suspect installs will be sped up as well.

@dhosterman dhosterman requested a review from wpietri April 30, 2024 14:43
@dhosterman dhosterman self-assigned this Apr 30, 2024
@github-actions
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Contributor

@wpietri wpietri left a comment

Choose a reason for hiding this comment

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

🥳

@dhosterman dhosterman merged commit cee8184 into main Apr 30, 2024
@dhosterman dhosterman deleted the remove-unnecessary-plugins branch April 30, 2024 17:02
@github-actions github-actions bot locked and limited conversation to collaborators Apr 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants