Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Dependency refactor, add tests#1165

Closed
lucylq wants to merge 1 commit into
pytorch:mainfrom
lucylq:export-D51036567
Closed

Dependency refactor, add tests#1165
lucylq wants to merge 1 commit into
pytorch:mainfrom
lucylq:export-D51036567

Conversation

@lucylq

@lucylq lucylq commented Nov 7, 2023

Copy link
Copy Markdown
Contributor

Summary:
Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

New dep chain:
op_.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

Old dep chain:
op_.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

Testing
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567

@pytorch-bot

pytorch-bot Bot commented Nov 7, 2023

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/1165

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 9d52379 with merge base bfebeda (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@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 Nov 7, 2023
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 8, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 8, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 8, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

1 similar comment
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 8, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 8, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 9, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 9, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 9, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 9, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 9, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Gated to only run with fbcode, as there is dependency on exir which is only in fbcode.

Reviewed By: larryliu0820

Differential Revision: D51036567
lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 10, 2023
Summary:

Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50

**New dep chain:**
op_<op>.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util

**Old dep chain:**
op_<op>.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h

**Testing**
Add e2e test with addmul model
Update unit test to use addmul model

Gated to only run with fbcode, as there is dependency on exir which is only in fbcode.

Reviewed By: larryliu0820

Differential Revision: D51036567
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in f4679ae.

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