Skip to content

Conversation

PaliC
Copy link
Contributor

@PaliC PaliC commented Aug 2, 2022

This change points multipy::runtime to use multipy.package instead of torch.package by copying _deploy.py (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow multipy::runtime to access multipy.package and _deploy.py.

@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 2, 2022
@PaliC PaliC changed the title Export d38142607 Use multipy.package in multipy/runtime Aug 2, 2022
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Aug 2, 2022

Codecov Report

Merging #111 (b0d994f) into main (bd528d0) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #111   +/-   ##
=======================================
  Coverage   88.86%   88.86%           
=======================================
  Files          65       65           
  Lines        3869     3869           
=======================================
  Hits         3438     3438           
  Misses        431      431           
Impacted Files Coverage Δ
multipy/package/__init__.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

@facebook-github-bot
Copy link
Contributor

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

PaliC added a commit that referenced this pull request Aug 2, 2022
Summary:
This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

Pull Request resolved: #111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: 0ca53ea57c3cb46fba70465cb671ffe7e9e68785
@facebook-github-bot
Copy link
Contributor

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

@PaliC PaliC force-pushed the export-D38142607 branch from 66a9713 to 498fc5d Compare August 2, 2022 23:25
Summary:
This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

Pull Request resolved: #111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: 8ec7316225284bd0adc2a9c9bdbe82c5d15bb295
PaliC added a commit to PaliC/pytorch that referenced this pull request Aug 2, 2022
Summary:
This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

X-link: meta-pytorch/multipy#111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: 44faa10a8b995b47f257d6fe20d9caa335237c9e
@PaliC PaliC force-pushed the export-D38142607 branch from 498fc5d to b0d994f Compare August 2, 2022 23:57
@facebook-github-bot
Copy link
Contributor

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

pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Aug 3, 2022
Summary:
This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

X-link: meta-pytorch/multipy#111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

Pull Request resolved: #82690
Approved by: https://github.com/d4l3k
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Aug 3, 2022
Summary:
Pull Request resolved: #82690

This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

X-link: meta-pytorch/multipy#111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: dec35e0f07efa1c989f6356dd073dfddd12a186e
@facebook-github-bot facebook-github-bot deleted the export-D38142607 branch August 7, 2022 14:18
anirbanr-fb-r2p pushed a commit to anirbanr-fb-r2p/multipy that referenced this pull request Aug 11, 2022
Summary:
X-link: pytorch/pytorch#82690

This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

Pull Request resolved: meta-pytorch#111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: dec35e0f07efa1c989f6356dd073dfddd12a186e
anirbanr-fb-r2p pushed a commit to anirbanr-fb-r2p/multipy that referenced this pull request Aug 11, 2022
Use multipy.package in `multipy/runtime` (#82690)

Summary:
X-link: pytorch/pytorch#82690

This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

Pull Request resolved: meta-pytorch#111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: dec35e0f07efa1c989f6356dd073dfddd12a186e
anirbanr-fb-r2p pushed a commit to anirbanr-fb-r2p/multipy that referenced this pull request Aug 11, 2022
Use multipy.package in `multipy/runtime` (#82690)

Summary:
X-link: pytorch/pytorch#82690

This change points multipy::runtime to use multipy.package instead of torch.package by copying `_deploy.py` (which is used in order to pass objects in and out of interpreters) into multipy as well as making the neccessary changes to allow `multipy::runtime` to access `multipy.package` and `_deploy.py`.

Pull Request resolved: meta-pytorch#111

Reviewed By: d4l3k

Differential Revision: D38337551

Pulled By: PaliC

fbshipit-source-id: dec35e0f07efa1c989f6356dd073dfddd12a186e
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants