Skip to content

Conversation

ljk53
Copy link
Contributor

@ljk53 ljk53 commented Mar 4, 2020

Stack from ghstack:

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

Differential Revision: D20267274

…ault

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

[ghstack-poisoned]
ljk53 added a commit that referenced this pull request Mar 4, 2020
…ault

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

ghstack-source-id: fd5d71a
Pull Request resolved: #34203
@ljk53 ljk53 requested review from IvanKobzarev, dreiss, dzhulgakov, ezyang and xta0 and removed request for xta0 March 4, 2020 08:04
build_args+=("USE_OBSERVERS=ON")
build_args+=("USE_ZSTD=ON")
"${ROOT_DIR}/scripts/build_android.sh" $(build_to_cmake ${build_args[@]}) "$@"
BUILD_CAFFE2_MOBILE=1 "${ROOT_DIR}/scripts/build_android.sh" $(build_to_cmake ${build_args[@]}) "$@"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still run this CI job? Could find it in .circleci/config.yml

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's still used, see caffe2_linux_build

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

caffe2_linux_build

right, but caffe2_linux_build doesn't seem to have any job with "${BUILD_ENVIRONMENT}" == *-android* (I meant to ask about the caffe2 android CI)

@dr-ci
Copy link

dr-ci bot commented Mar 4, 2020

💊 CircleCI build failures summary and remediations

As of commit 9ff683b (more details on the Dr. CI page):


None of the build failures appear to be your fault 💚



❄️ 1 tentatively flaky failure

1 failure tentatively classified as flaky but have not launched reruns to confirm:

See CircleCI build caffe2_onnx_py2_gcc5_ubuntu16_04_test (1/1)

Step: "Test" (full log | pattern match details) ❄️

Mar 06 02:48:38 E RuntimeError: required keyword attribute 'Y_scale' is undefined
Mar 06 02:48:38 k = 'Y_scale' 
Mar 06 02:48:38  
Mar 06 02:48:38     def _node_getitem(self, k): 
Mar 06 02:48:38         r""" 
Mar 06 02:48:38         Accessor for attributes of a node which is polymorphic over 
Mar 06 02:48:38         return type. 
Mar 06 02:48:38      
Mar 06 02:48:38         NB: This is monkey-patched onto Node. 
Mar 06 02:48:38         """ 
Mar 06 02:48:38 >       sel = self.kindOf(k) 
Mar 06 02:48:38 E       RuntimeError: required keyword attribute 'Y_scale' is undefined 
Mar 06 02:48:38  
Mar 06 02:48:38 ../.local/lib/python2.7/site-packages/torch/onnx/utils.py:869: RuntimeError 
Mar 06 02:48:38 =============================== warnings summary =============================== 
Mar 06 02:48:38 /usr/local/lib/python2.7/dist-packages/scipy/_lib/_numpy_compat.py:10 
Mar 06 02:48:38   /usr/local/lib/python2.7/dist-packages/scipy/_lib/_numpy_compat.py:10: DeprecationWarning: Importing from numpy.testing.nosetester is deprecated since 1.15.0, import from numpy.testing instead. 
Mar 06 02:48:38     from numpy.testing.nosetester import import_nose 
Mar 06 02:48:38  
Mar 06 02:48:38 /usr/local/lib/python2.7/dist-packages/scipy/stats/morestats.py:12 
Mar 06 02:48:38   /usr/local/lib/python2.7/dist-packages/scipy/stats/morestats.py:12: DeprecationWarning: Importing from numpy.testing.decorators is deprecated since numpy 1.15.0, import from numpy.testing instead. 
Mar 06 02:48:38     from numpy.testing.decorators import setastest 

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

This comment has been revised 19 times.

…orch by default"

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

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

[ghstack-poisoned]
ljk53 added a commit that referenced this pull request Mar 5, 2020
…ault

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

ghstack-source-id: e58ab26
Pull Request resolved: #34203
@ezyang
Copy link
Contributor

ezyang commented Mar 5, 2020

Pretty sure it's Android only, but maybe @pietern would know better

…orch by default"

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

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

[ghstack-poisoned]
…orch by default"

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

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

[ghstack-poisoned]
ljk53 added a commit that referenced this pull request Mar 6, 2020
…ault

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

ghstack-source-id: 38f0f4b
Pull Request resolved: #34203
…orch by default"

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

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

[ghstack-poisoned]
ljk53 added a commit that referenced this pull request Mar 6, 2020
…ault

Summary:
Currently cmake and mobile build scripts still build libcaffe2 by
default. To build pytorch mobile users have to set environment variable
BUILD_PYTORCH_MOBILE=1 or set cmake option BUILD_CAFFE2_MOBILE=OFF.

PyTorch mobile has been released for a while. It's about time to change
CMake and build scripts to build libtorch by default.

Changed caffe2 CI job to build libcaffe2 by setting BUILD_CAFFE2_MOBILE=1
environment variable. Only found android CI for libcaffe2 - do we ever
have iOS CI for libcaffe2?

ghstack-source-id: 3ec7b18
Pull Request resolved: #34203
@facebook-github-bot
Copy link
Contributor

@ljk53 merged this pull request in 9a5e9d8.

@facebook-github-bot facebook-github-bot deleted the gh/ljk53/116/head branch March 9, 2020 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants