Skip to content

structural api and schema update #410

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

Conversation

Gasoonjia
Copy link
Contributor

Summary:
Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D49406613

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

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

@facebook-github-bot
Copy link
Contributor

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

Songhao Jia and others added 3 commits September 21, 2023 11:37
Summary:
Pull Request resolved: pytorch/executorch#396

This diff creates a unique namespace for bundled program from core executorch schema. Such update can make bundled program more decomposed from core ET.

Differential Revision: https://internalfb.com/D49326299

fbshipit-source-id: 259480d01795626ef7ea2074955de509b2ed3b9e
Summary:
Pull Request resolved: pytorch/executorch#397

Now we already have a specific namespace for bundledprogram, this diff removes the "Bundled" prefix for part of bundled program schema's variables.
Rest variables will be updated in the following diffs.

Differential Revision: https://internalfb.com/D49337170

fbshipit-source-id: a855f6296650ed5008a412938b8492d161454cf2
Summary:
Pull Request resolved: pytorch/executorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D49406613

fbshipit-source-id: 3741c769ea84105b1e64c0f1e6503b6903b91057
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

4 similar comments
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 18, 2023
Summary:

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 18, 2023
Summary:

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: a1154a19514324019e1ec7f590ed6a19583ecf99
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: d23c842acb105b34436764bf346effc2af8ae9da
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: fc65f20001a6925e981263b64a9fa8a9c21bd9e2
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 18, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: 1f7331c4463ea6c4ad4e5a93a81d570633711062
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 19, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: 7e1a9429226f27fe18f9d80cc434e0ac6a6231c6
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 19, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: 2e5293034c4737be2a72fc9565eade984636d87e
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 19, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: c347b50578bd2bdcd7e9ef81e68849e046876c70
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 20, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: e379b9cb7da0ae9fb97252d4c52aaa23894feac2
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 20, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: 92ecd69a6c43bbdb78585a596ccdd9ca916f8b9f
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 20, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: 096ad8e35f86937babeb9a9a518c5b81b3e1f2e1
Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request Oct 20, 2023
Summary:
Pull Request resolved: pytorch#410

Update the AOT API for bundled program to make it more user friendly.

We created two new classes for holding testcases in a more structural way:

MethodTestCase: contains inputs and expected outputs for a single test for method.

MethodTestSuite: Collection of all test cases for a program method.

Detailed design can be found here: https://docs.google.com/document/d/170WJ81dPuNQsvsjqH_wQjafbu2kxtEG1sgX-z6OqCEI/edit#heading=h.69fgezruwl6n

Differential Revision: D50422563

fbshipit-source-id: e0f8f6438df4af593f1cc135a39da4a8860174a8
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants