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.

Sign up for GitHub

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

Jump to bottom

[refactor][PL][9/N] change the name of the _load_checkpoint to be more descriptive to contain test #1030

Closed

hackgoofer wants to merge 21 commits into gh/ytsheng/17/base from gh/ytsheng/17/head

Contributor

hackgoofer commented Jul 23, 2021 •

edited

Stack from ghstack:

[feat][PL][11/N] Inference with test #1035 [feat][PL][11/N] Inference with test
[feat][PL][10/N] lightning distributed #1033 [feat][PL][10/N] lightning distributed
[refactor][PL][9/N] change the name of the _load_checkpoint to be more descriptive to contain test #1030 [refactor][PL][9/N] change the name of the _load_checkpoint to be more descriptive to contain test
[feat][PL][8/N] save config with checkpoint #1024 [feat][PL][8/N] save config with checkpoint
[feat][PL][7/N] trainer checkpoint from mmf to lightning update #1023 [feat][PL][7/N] trainer checkpoint from mmf to lightning update
[feat][PL][6/N] pretrained state mapping for PL #1022 [feat][PL][6/N] pretrained state mapping for PL

Differential Revision: D29888294


          [refactor] change the name of the _load_checkpoint to be more descrip…

b84b57e

…tive to contain test

[ghstack-poisoned]

This was referenced Jul 23, 2021

[fix][PL][1/N]: clean up lightning test code #1017

Closed

[feat][PL][2/N] PL train checkpoint #1018

Closed

facebook-github-bot added the CLA Signed label

This was referenced Jul 23, 2021

[fix][PL][3/N] minor change in base model #1019

Closed

[feat][PL][4/N] best ckpt #1020

Closed

[feat][PL][5/N] load model checkpoint directly + model zoo #1021

Closed

[feat][PL][6/N] pretrained state mapping for PL #1022

Closed

[feat][PL][7/N] trainer checkpoint from mmf to lightning update #1023

Closed

[feat][PL][8/N] save config with checkpoint #1024

Closed

hackgoofer pushed a commit that referenced this pull request


          [refactor] change the name of the _load_checkpoint to be more descrip…

42cb402

…tive to contain test

ghstack-source-id: bc7551543ad27405e8ac20cbd21849be6ac41428
Pull Request resolved: #1030

hackgoofer changed the title ~~[refactor] change the name of the _load_checkpoint to be more descriptive to contain test~~ [refactor][PL][9/N] change the name of the _load_checkpoint to be more descriptive to contain test

Contributor Author

hackgoofer commented Jul 23, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

02d7914

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

hackgoofer mentioned this pull request

[feat][PL][10/N] lightning distributed #1032

Closed

Contributor

facebook-github-bot commented Jul 29, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

hackgoofer mentioned this pull request

[feat][PL][10/N] lightning distributed #1033

Closed


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

75e60cf

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Jul 29, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Jul 29, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

4f11523

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Jul 29, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Jul 29, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

25cea7d

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

hackgoofer mentioned this pull request

[feat][PL][11/N] Inference with test #1035

Closed

Contributor

facebook-github-bot commented Jul 29, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Jul 29, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

ffec27f

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Jul 30, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

ae1df6e

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Aug 30, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Aug 30, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

e5b7030

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Aug 31, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Aug 31, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

c27624a

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 7, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Sep 7, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

a151dff

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 7, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Sep 7, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

3db2e6f

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 7, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Sep 7, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

ae93de4

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

hackgoofer mentioned this pull request

[fix] demjson depends on Setuptools 58.0.0, which has removed support for 2to3, so import would fail #1076

Closed

Contributor

facebook-github-bot commented Sep 8, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Sep 8, 2021

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


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

7dc82c0

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 8, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

e175dd4

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 8, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

792bfcd

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 8, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.


          Update on "[refactor][PL][9/N] change the name of the _load_checkpoin…

a30acfb

…t to be more descriptive to contain test"

Differential Revision: [D29888294](https://our.internmc.facebook.com/intern/diff/D29888294)

[ghstack-poisoned]

Contributor

facebook-github-bot commented Sep 8, 2021

@ytsheng has updated the pull request. You must reimport the pull request before landing.

Contributor Author

hackgoofer commented Sep 8, 2021

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

facebook-github-bot closed this

facebook-github-bot pushed a commit that referenced this pull request


          [refactor][PL][9/N] change the name of the _load_checkpoint to be mor…

15fa630

…e descriptive to contain test (#1030)

Summary: Pull Request resolved: #1030

Test Plan: Imported from OSS

Reviewed By: simran2905

Differential Revision: D29888294

Pulled By: ytsheng

fbshipit-source-id: 54919dc4e8448a1240b2cc72a364702aa7cf49c2

apsdehal deleted the gh/ytsheng/17/head branch

October 5, 2021 22:25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment