Skip to content

Add timm package to CI #88

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

Closed
wants to merge 3 commits into from
Closed

Add timm package to CI #88

wants to merge 3 commits into from

Conversation

kirklandsign
Copy link
Contributor

This is required for some models like inception v4

This is required for some models like inception v4
@kirklandsign kirklandsign requested a review from huydhn August 22, 2023 01:32
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 22, 2023
@huydhn
Copy link
Contributor

huydhn commented Aug 22, 2023

Do you really need to have timm installed via a pinned commit? Because if you could get it from pypi https://pypi.org/project/timm/#history, you could simply add timm==0.6.13 into https://github.com/pytorch/executorch/blob/main/.ci/docker/requirements-ci.txt like a regular dependency and the CI will do the rest (for bot linux and macos)

@facebook-github-bot
Copy link
Contributor

@kirklandsign has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@kirklandsign merged this pull request in 4885608.

jirioc pushed a commit to nxp-upstream/executorch that referenced this pull request May 14, 2025
…EFORE_FULLY_CONNECTED_WITH_SUM

Merge in AITEC/executorch from feature/nxg11066/EIEX-194-turn-off-optimization-ReplaceAveragePoolBeforeFullyConnectedWithSum to main-nxp

* commit '5ef87a69f050c257230779421e87eddd666cd8ec':
  Remove optimization REPLACE_AVERAGE_POOL_BEFORE_FULLY_CONNECTED_WITH_SUM
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants