Skip to content

Add aten::squeeze and aten::unsqueeze to functionalization #274

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 1 commit into from
Closed

Add aten::squeeze and aten::unsqueeze to functionalization #274

wants to merge 1 commit into from

Conversation

dulinriley
Copy link
Contributor

Summary:
aten::squeeze and aten::unsqueeze only have out variants defined for the functional
variants: squeeze_copy and unsqueeze_copy.

Add these to a conversion pass similar to permute and transpose.

Differential Revision: D49171160

@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 Sep 12, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49171160

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49171160

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49171160

Summary:

`aten::squeeze` and `aten::unsqueeze` only have out variants defined for the functional
variants: `squeeze_copy` and `unsqueeze_copy`.

Add these to a conversion pass similar to permute and transpose.

Reviewed By: jameshegarty

Differential Revision: D49171160
@netlify
Copy link

netlify bot commented Oct 3, 2023

Deploy Preview for resplendent-gnome-14e531 canceled.

Name Link
🔨 Latest commit 4846e29
🔍 Latest deploy log https://app.netlify.com/sites/resplendent-gnome-14e531/deploys/651c9e80793a41000878dd4f

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49171160

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49171160

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in f4e1692.

@dulinriley dulinriley deleted the export-D49171160 branch October 13, 2023 21:33
Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
Summary:
This enables us to benchmark models on android

Test Plan:
export ANDROID_NDK="path-to"
./runner-et/build_android.sh

adb push build/cmake-out-android/runner_et /data/local/tmp/
adb push /tmp/stories110m_a8w4dq.pte /data/local/tmp/
adb push /tmp/tokenizer.bin /data/local/tmp/
adb shell "cd /data/local/tmp/ && runner_et stories110m_fp32.pte -z tokenizer.bin  -n 200 -t 0"
produced text
Once upon a time, there was a little girl named Lily. She loved to play
outside in the sunshine.

Reviewers:

Subscribers:

Tasks:

Tags:
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. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants