Skip to content

Commit b0ad11a

Browse files
HDCharlesHDCharles
committed
Update on "[ao][fx] fixing public v private graph_module.py"
Summary: made _is_observed_module, _is_observed_standalone_module private Test Plan: python test/test_public_bindings.py Reviewers: Subscribers: Tasks: Tags: Differential Revision: [D41015545](https://our.internmc.facebook.com/intern/diff/D41015545) cc jerryzh168 jianyuh raghuramank100 jamesr66a vkuzo jgong5 Xia-Weiwen leslie-fang-intel [ghstack-poisoned]
2 parents cd3bd5d + 5cdcd83 commit b0ad11a

File tree

248 files changed

+5220
-2341
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

248 files changed

+5220
-2341
lines changed

‎.circleci/scripts/binary_ios_upload.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ fi
3333
cp ${PROJ_ROOT}/LICENSE ${ZIP_DIR}/
3434
# zip the library
3535
export DATE="$(date -u +%Y%m%d)"
36-
export IOS_NIGHTLY_BUILD_VERSION="1.14.0.${DATE}"
36+
export IOS_NIGHTLY_BUILD_VERSION="2.0.0.${DATE}"
3737
if [ "${BUILD_LITE_INTERPRETER}" == "1" ]; then
3838
# libtorch_lite_ios_nightly_1.11.0.20210810.zip
3939
ZIPFILE="libtorch_lite_ios_nightly_${IOS_NIGHTLY_BUILD_VERSION}.zip"

‎.circleci/scripts/binary_populate_env.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ PIP_UPLOAD_FOLDER='nightly/'
5959
# We put this here so that OVERRIDE_PACKAGE_VERSION below can read from it
6060
export DATE="$(date -u +%Y%m%d)"
6161
#TODO: We should be pulling semver version from the base version.txt
62-
BASE_BUILD_VERSION="1.14.0.dev$DATE"
62+
BASE_BUILD_VERSION="2.0.0.dev$DATE"
6363
# Change BASE_BUILD_VERSION to git tag when on a git tag
6464
# Use 'git -C' to make doubly sure we're in the correct directory for checking
6565
# the git tag

‎.flake8‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,11 @@ ignore =
1212
B007,B008,
1313
# these ignores are from flake8-comprehensions; please fix!
1414
C400,C401,C402,C403,C404,C405,C407,C411,C413,C414,C415
15-
per-file-ignores = __init__.py: F401 torch/utils/cpp_extension.py: B950
15+
per-file-ignores =
16+
__init__.py: F401
17+
torch/utils/cpp_extension.py: B950
18+
torchgen/api/types/__init__.py: F401,F403
19+
torchgen/executorch/api/types/__init__.py: F401,F403
1620
optional-ascii-coding = True
1721
exclude =
1822
./.git,

‎.github/ci_commit_pins/vision.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
842e178a488722720b6eb1e9cb508439e8e1ecd9
1+
677fc939b21a8893f07db4c1f90482b648b6573f

‎.github/ci_commit_pins/xla.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
b55aec841b9cf680b04abefaf3c0197a51de8b08
1+
e5b42e75340d62f06e0667a3750ed99d46f0b852

‎.github/merge_rules.yaml‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
- name: ONNX exporter
22
patterns:
33
- .jenkins/caffe2/*
4+
- .jenkins/onnx/*
45
- aten/src/ATen/core/interned_strings.h
56
- docs/source/onnx.rst
67
- docs/source/onnx*
@@ -20,6 +21,11 @@
2021
approved_by:
2122
- BowenBao
2223
- abock
24+
- justinchuby
25+
- shubhambhokare1
26+
- thiagocrepaldi
27+
- titaiwangms
28+
- wschin
2329
mandatory_checks_name:
2430
- EasyCLA
2531
- Lint

‎.github/workflows/_linux-test.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ jobs:
133133
if [[ $TEST_CONFIG == 'multigpu' ]]; then
134134
TEST_COMMAND=.jenkins/pytorch/multigpu-test.sh
135135
elif [[ $BUILD_ENVIRONMENT == *onnx* ]]; then
136-
TEST_COMMAND=.jenkins/caffe2/test.sh
136+
TEST_COMMAND=.jenkins/onnx/test.sh
137137
else
138138
TEST_COMMAND=.jenkins/pytorch/test.sh
139139
fi

‎.github/workflows/nightly.yml‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ jobs:
3838

3939
update-vision-commit-hash:
4040
uses: ./.github/workflows/_update-commit-hash.yml
41+
if: ${{ github.event_name == 'schedule' }}
4142
with:
4243
repo-name: vision
4344
branch: main

‎.jenkins/caffe2/test.sh‎

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -170,19 +170,3 @@ if [[ "${SHARD_NUMBER:-1}" == "1" ]]; then
170170
"$caffe2_pypath/python" \
171171
"${EXTRA_TESTS[@]}"
172172
fi
173-
174-
##############
175-
# ONNX tests #
176-
##############
177-
if [[ "$BUILD_ENVIRONMENT" == *onnx* ]]; then
178-
pip install -q --user --no-use-pep517 "git+https://github.com/pytorch/vision.git@$(cat .github/ci_commit_pins/vision.txt)"
179-
pip install -q --user ninja flatbuffers==2.0 numpy==1.21.5 onnxruntime==1.12.1 beartype==0.10.4 onnx==1.12.0
180-
# TODO: change this when onnx-script is on testPypi
181-
pip install 'onnx-script @ git+https://github.com/microsoft/onnx-script'
182-
# numba requires numpy <= 1.20, onnxruntime requires numpy >= 1.21.
183-
# We don't actually need it for our tests, but it's imported if it's present, so uninstall.
184-
pip uninstall -q --yes numba
185-
# JIT C++ extensions require ninja, so put it into PATH.
186-
export PATH="/var/lib/jenkins/.local/bin:$PATH"
187-
"$ROOT_DIR/scripts/onnx/test.sh"
188-
fi

‎.jenkins/onnx/README.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# Jenkins
2+
3+
The scripts in this directory are the entrypoint for testing ONNX exporter.
4+
5+
The environment variable `BUILD_ENVIRONMENT` is expected to be set to
6+
the build environment you intend to test. It is a hint for the build
7+
and test scripts to configure Caffe2 a certain way and include/exclude
8+
tests. Docker images, they equal the name of the image itself. For
9+
example: `py2-cuda9.0-cudnn7-ubuntu16.04`. The Docker images that are
10+
built on Jenkins and are used in triggered builds already have this
11+
environment variable set in their manifest. Also see
12+
`./docker/jenkins/*/Dockerfile` and search for `BUILD_ENVIRONMENT`.
13+
14+
Our Jenkins installation is located at https://ci.pytorch.org/jenkins/.

0 commit comments

Comments
 (0)