Skip to content
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

[quant][fix] Fix quant type classification for float_qparam qconfig #48069

Closed
wants to merge 8 commits into from

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Nov 17, 2020

Stack from ghstack:

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D25010175

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Nov 17, 2020

💊 CI failures summary and remediations

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


  • 3/3 failures introduced in this PR

🕵️ 3 new failures recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

See CircleCI build pytorch_linux_xenial_cuda9_2_cudnn7_py3_gcc5_4_build (1/3)

Step: "Build" (full log | diagnosis details | 🔁 rerun)

Nov 18 01:09:51 /var/lib/jenkins/workspace/torch/csrc/jit/mobile/import.h:16:32: error: unknown option after '#pragma GCC diagnostic' kind [-Werror=pragmas]
Nov 18 01:09:46 [ 75%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/codegen/fuser/cpu/fused_kernel.cpp.o 
Nov 18 01:09:46 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/api/module_save.cpp.o 
Nov 18 01:09:47 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/utils/byte_order.cpp.o 
Nov 18 01:09:47 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/serialization/import_legacy.cpp.o 
Nov 18 01:09:48 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/autograd.cpp.o 
Nov 18 01:09:49 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/utils.cpp.o 
Nov 18 01:09:49 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/context/container.cpp.o 
Nov 18 01:09:50 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/context/context.cpp.o 
Nov 18 01:09:51 [ 76%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/engine/dist_engine.cpp.o 
Nov 18 01:09:51 In file included from /var/lib/jenkins/workspace/torch/csrc/jit/mobile/import.cpp:1:0: 
Nov 18 01:09:51 /var/lib/jenkins/workspace/torch/csrc/jit/mobile/import.h:16:32: error: unknown option after '#pragma GCC diagnostic' kind [-Werror=pragmas] 
Nov 18 01:09:51  #pragma GCC diagnostic ignored "-Wglobal-constructors" 
Nov 18 01:09:51                                 ^ 
Nov 18 01:09:51 cc1plus: all warnings being treated as errors 
Nov 18 01:09:51 make[2]: *** [caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/mobile/import.cpp.o] Error 1 
Nov 18 01:09:51 make[2]: *** Waiting for unfinished jobs.... 
Nov 18 01:09:51 caffe2/CMakeFiles/torch_cpu.dir/build.make:25454: recipe for target 'caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/mobile/import.cpp.o' failed 
Nov 18 01:09:52 make[1]: *** [caffe2/CMakeFiles/torch_cpu.dir/all] Error 2 
Nov 18 01:09:52 CMakeFiles/Makefile2:9923: recipe for target 'caffe2/CMakeFiles/torch_cpu.dir/all' failed 
Nov 18 01:09:52 Makefile:138: recipe for target 'all' failed 
Nov 18 01:09:52 make: *** [all] Error 2 

See CircleCI build binary_linux_libtorch_3_7m_cpu_gcc5_4_cxx11-abi_shared-with-deps_build (2/3)

Step: "Build" (full log | diagnosis details | 🔁 rerun)

RuntimeError: no cmake or cmake3 with version >= 3.5.0 found
++ [[ patchelf 0.12 == \p\a\t\c\h\e\l\f\ \0\.\9 ]] 
++ python setup.py clean 
Building wheel torch-1.8.0.dev20201118+cpu 
Traceback (most recent call last): 
  File "setup.py", line 284, in <module> 
    cmake = CMake() 
  File "/pytorch/tools/setup_helpers/cmake.py", line 98, in __init__ 
    self._cmake_command = CMake._get_cmake_command() 
  File "/pytorch/tools/setup_helpers/cmake.py", line 125, in _get_cmake_command 
    raise RuntimeError('no cmake or cmake3 with version >= 3.5.0 found') 
RuntimeError: no cmake or cmake3 with version >= 3.5.0 found 

See CircleCI build pytorch_linux_xenial_py3_6_gcc5_4_build (3/3)

Step: "Build" (full log | diagnosis details | 🔁 rerun)

Nov 18 01:04:42 /var/lib/jenkins/workspace/torch/csrc/jit/mobile/import.h:16:32: error: unknown option after '#pragma GCC diagnostic' kind [-Werror=pragmas]
Nov 18 01:04:37 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/utils/byte_order.cpp.o 
Nov 18 01:04:38 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/serialization/import_legacy.cpp.o 
Nov 18 01:04:38 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/autograd.cpp.o 
Nov 18 01:04:39 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/utils.cpp.o 
Nov 18 01:04:39 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/context/container.cpp.o 
Nov 18 01:04:40 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/context/context.cpp.o 
Nov 18 01:04:40 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/engine/dist_engine.cpp.o 
Nov 18 01:04:41 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/functions/recvrpc_backward.cpp.o 
Nov 18 01:04:41 [ 85%] Building CXX object caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/distributed/autograd/functions/sendrpc_backward.cpp.o 
Nov 18 01:04:42 In file included from /var/lib/jenkins/workspace/torch/csrc/jit/mobile/import.cpp:1:0: 
Nov 18 01:04:42 /var/lib/jenkins/workspace/torch/csrc/jit/mobile/import.h:16:32: error: unknown option after '#pragma GCC diagnostic' kind [-Werror=pragmas] 
Nov 18 01:04:42  #pragma GCC diagnostic ignored "-Wglobal-constructors" 
Nov 18 01:04:42                                 ^ 
Nov 18 01:04:42 cc1plus: all warnings being treated as errors 
Nov 18 01:04:42 caffe2/CMakeFiles/torch_cpu.dir/build.make:25406: recipe for target 'caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/mobile/import.cpp.o' failed 
Nov 18 01:04:42 make[2]: *** [caffe2/CMakeFiles/torch_cpu.dir/__/torch/csrc/jit/mobile/import.cpp.o] Error 1 
Nov 18 01:04:42 make[2]: *** Waiting for unfinished jobs.... 
Nov 18 01:04:42 make[1]: *** [caffe2/CMakeFiles/torch_cpu.dir/all] Error 2 
Nov 18 01:04:42 CMakeFiles/Makefile2:8620: recipe for target 'caffe2/CMakeFiles/torch_cpu.dir/all' failed 
Nov 18 01:04:42 Makefile:138: recipe for target 'all' failed 
Nov 18 01:04:42 make: *** [all] Error 2 

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 or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 42 times.

…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Nov 17, 2020
Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 59ab7a6325ced695d826488a69babb92595dd6f3
Pull Request resolved: #48069
…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
…m qconfig"

Summary:
also renamed float_qparam_dynamic_qconfig to float_qparam_weight_only_qconfig
It's not used in user code yet so we only need to update the tests.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 576fa09.

@facebook-github-bot facebook-github-bot deleted the gh/jerryzh168/507/head branch November 22, 2020 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants