Skip to content

Conversation

datumbox
Copy link
Contributor

@datumbox datumbox commented Feb 2, 2022

Adding the prototype flag on quantization reference script similar to what we did for other scripts on #5248

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Feb 2, 2022

💊 CI failures summary and remediations

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



🕵️ 2 new failures recognized by patterns

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

See GitHub Actions build CodeQL / build (1/2)

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

2022-02-02T10:11:52.8212269Z ##[error]Process completed with exit code 1.
2022-02-02T10:11:52.6633680Z     self.finalize_options()
2022-02-02T10:11:52.6634275Z   File "/home/runner/.local/lib/python3.8/site-packages/setuptools/command/develop.py", line 52, in finalize_options
2022-02-02T10:11:52.6634771Z     easy_install.finalize_options(self)
2022-02-02T10:11:52.6635445Z   File "/home/runner/.local/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 276, in finalize_options
2022-02-02T10:11:52.6635929Z     self._fix_install_dir_for_user_site()
2022-02-02T10:11:52.6636625Z   File "/home/runner/.local/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 382, in _fix_install_dir_for_user_site
2022-02-02T10:11:52.6637080Z     self.create_home_path()
2022-02-02T10:11:52.6637970Z   File "/home/runner/.local/lib/python3.8/site-packages/setuptools/command/easy_install.py", line 1338, in create_home_path
2022-02-02T10:11:52.6638522Z     if path.startswith(home) and not os.path.isdir(path):
2022-02-02T10:11:52.6677234Z AttributeError: 'int' object has no attribute 'startswith'
2022-02-02T10:11:52.8212269Z ##[error]Process completed with exit code 1.
2022-02-02T10:11:52.8295139Z Post job cleanup.
2022-02-02T10:11:52.9729435Z [command]/usr/bin/git version
2022-02-02T10:11:52.9803949Z git version 2.35.1
2022-02-02T10:11:52.9843552Z [command]/usr/bin/git config --local --name-only --get-regexp core\.sshCommand
2022-02-02T10:11:52.9901339Z [command]/usr/bin/git submodule foreach --recursive git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :
2022-02-02T10:11:53.0367608Z [command]/usr/bin/git config --local --name-only --get-regexp http\.https\:\/\/github\.com\/\.extraheader
2022-02-02T10:11:53.0410620Z http.https://github.com/.extraheader
2022-02-02T10:11:53.0423328Z [command]/usr/bin/git config --local --unset-all http.https://github.com/.extraheader
2022-02-02T10:11:53.0477380Z [command]/usr/bin/git submodule foreach --recursive git config --local --name-only --get-regexp 'http\.https\:\/\/github\.com\/\.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :
2022-02-02T10:11:53.1100157Z Cleaning up orphan processes

See CircleCI build unittest_prototype (2/2)

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

/home/circleci/project/test/test_prototype_feat...ead to errors or silently give incorrect results).
    new_mod_list = fuser_func(mod_list, is_qat, additional_fuser_method_mapping)
  File "/home/circleci/.local/lib/python3.7/site-packages/torch/ao/quantization/fuse_modules.py", line 49, in fuse_known_modules
    fused = fuser_method(is_qat, *mod_list)
  File "/home/circleci/.local/lib/python3.7/site-packages/torch/ao/quantization/fuser_method_mappings.py", line 46, in fuse_conv_bn
    return nn.utils.fuse_conv_bn_eval(conv, bn)
  File "/home/circleci/.local/lib/python3.7/site-packages/torch/nn/utils/fusion.py", line 7, in fuse_conv_bn_eval
    assert(not (conv.training or bn.training)), "Fusion only for eval!"
AssertionError: Fusion only for eval!
�[33m=============================== warnings summary ===============================�[0m
test/test_prototype_features.py::TestJit::test_bounding_box
  /home/circleci/project/test/test_prototype_features.py:150: TracerWarning: Iterating over a tensor might cause the trace to be incorrect. Passing a tensor of different shape won't change the number of iterations executed (and might lead to errors or silently give incorrect results).
    new_height, new_width = size

test/test_prototype_features.py::TestJit::test_bounding_box
  /home/circleci/project/torchvision/prototype/features/_feature.py:78: TracerWarning: torch.as_tensor results are registered as constants in the trace. You can safely ignore this warning if you use this function to create tensors out of constant variables that would be the same every time you call this function. In any other case, this might cause the trace to be incorrect.
    return torch.as_tensor(data, dtype=dtype, device=device)

test/test_prototype_features.py::TestJit::test_bounding_box
  /home/circleci/project/test/test_prototype_features.py:164: TracerWarning: Converting a tensor to a Python list might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
    new_x1, new_y1, new_x2, new_y2, like=input, format="xyxy", image_size=tuple(size.tolist())


1 failure not recognized by patterns:

Job Step Action
CircleCI cmake_macos_cpu curl -o conda.sh https://repo.anaconda.com/miniconda/Miniconda3-latest-MacOSX-x86_64.sh
sh conda.sh -b
source $HOME/miniconda3/bin/activate
conda install -yq conda-build cmake
packaging/build_cmake.sh
🔁 rerun

🚧 3 ongoing upstream failures:

These were probably caused by upstream breakages that are not fixed yet.


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@datumbox datumbox merged commit 61a52b9 into pytorch:main Feb 2, 2022
@datumbox datumbox deleted the references/add_prototype_in_quant branch February 2, 2022 10:09
facebook-github-bot pushed a commit that referenced this pull request Feb 4, 2022
Summary: adding prototype flag.

Reviewed By: kazhang

Differential Revision: D33995923

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

3 participants