19 Commits

Author SHA1 Message Date
bcf50636ba [CI] Removing --user flag from all pip install commands (#154900)
Related to https://github.com/pytorch/pytorch/issues/148335

python virtualenv doesn't support using `--user` flag:

```
ERROR: Can not perform a '--user' install. User site-packages are not visible in this virtualenv.
+ python3 -m pip install --progress-bar off --user ninja==1.10.2
```

Pull Request resolved: https://github.com/pytorch/pytorch/pull/154900
Approved by: https://github.com/jeffdaily

Co-authored-by: Jithun Nair <jithun.nair@amd.com>
2025-07-14 21:09:42 +00:00
7d39e73c57 Fix more URLs (#153277)
Or ignore them.
Found by running the lint_urls.sh script locally with https://github.com/pytorch/pytorch/pull/153246

Pull Request resolved: https://github.com/pytorch/pytorch/pull/153277
Approved by: https://github.com/malfet
2025-05-14 16:23:50 +00:00
a13cf5d396 Workaround dind-rootless bind mount permissions (#123641)
ARC uses dind-rootless which causes bind mounts to always be mounted as the "root" user inside the container rather than the "jenkins" user as expected. We run chown to ensure that the workspace gets mapped to the jenkins user as well as a trap to ensure this change gets reverted when the script ends for any reason. This is the same workaround as in #122922 but adapted for onnx tests.

Issue: pytorch/ci-infra#112

Pull Request resolved: https://github.com/pytorch/pytorch/pull/123641
Approved by: https://github.com/jeanschmidt, https://github.com/seemethere
2024-04-10 22:44:57 +00:00
1e260c851b [ez] Don't retry onnx in shell (#113803)
is this important? not really, but the retries given by run_test.py on it's own should be enough
Pull Request resolved: https://github.com/pytorch/pytorch/pull/113803
Approved by: https://github.com/BowenBao
2023-11-15 23:45:25 +00:00
00ee38c661 [ONNX] Export module as function (#105618)
Introduce `Modularize` pass that analyzes the flat `fx.GraphModule` and creates nested
layers of sub `fx.GraphModule`s along with the `call_module` fx nodes that invokes them.
The analysis is done on the meta data "nn_module_stack", which captures the `nn.Module`
each flat `fx.Node` belongs to.

`FxOnnxInterpreter` is updated to support `call_module`. The related sub module linked
by `node.target` is exported as an ONNX model local function. The `call_module` node itself
is exported as an ONNX node, associated with the ONNX model local function by op_type.
Pull Request resolved: https://github.com/pytorch/pytorch/pull/105618
Approved by: https://github.com/justinchuby
2023-07-25 03:28:31 +00:00
0a0f107b50 Retry ONNX tests (the quick way) (#98627)
This is to mitigate a flaky ONNX test in trunk and also improve its reliability till we have https://github.com/pytorch/pytorch/issues/98626  (I figure that this is better than moving the job to unstable).

I try to disable the flaky test https://github.com/pytorch/pytorch/issues/98622, but that won't work as @clee2000 points out because ONNX isn't part of `run_test.py` to download and apply the list of disabled tests.
Pull Request resolved: https://github.com/pytorch/pytorch/pull/98627
Approved by: https://github.com/BowenBao
2023-04-07 22:20:39 +00:00
be0b415a5a [ONNX] Set shape/type into torchscript (#96349)
Fixes https://github.com/pytorch/pytorch/pull/95676#issuecomment-1460588229

PS: It doesn't seem the exported ONNX_proto having type now. I wonder if there was a ONNX pass doing this for us (converting torch dtype to onnx dtype during exporting.)

Type promotion issue would be raised with an error if we want to set type
```python
onnxscript_value.dtype = expected_value.dtype
```
onnx.onnx_cpp2py_export.shape_inference.InferenceError: [ShapeInferenceError] Shape inference error(s): (op_type:aten_add, node name: aten_add_1): [ShapeInferenceError] (op_type:Add, node name: n3): B has inconsistent type tensor(int64)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/96349
Approved by: https://github.com/justinchuby, https://github.com/wschin
2023-03-19 21:58:10 +00:00
6a2dcfd738 Move all ONNX test dependencies to Docker (#96590)
Per title.  This is the first one of a two-part process:

[x] Move all ONNX test dependencies to Docker https://github.com/pytorch/pytorch/pull/96590
[ ] Move the test model used by [TestFxToOnnxWithOnnxRuntime.test_gpt2_tiny](https://hud.pytorch.org/failure/FAILED%20test%2Fonnx%2Ftest_fx_to_onnx_with_onnxruntime.py%3A%3ATestFxToOnnxWithOnnxRuntime%3A%3Atest_large_scale_exporter_with_tiny_gpt2%20-%20requests.exceptions.ReadTimeout%3A%20HTTPSConnectionPool(host%3D'huggingface.co'%2C%20port%3D443)%3A%20Read%20timed%20out.%20(read%20timeout%3D10.0))
Pull Request resolved: https://github.com/pytorch/pytorch/pull/96590
Approved by: https://github.com/ZainRizvi
2023-03-14 06:19:00 +00:00
df0ff34bcb [ONNX] Bump onnx submodule to release 1.13.1 from rc2 (#96325)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/96325
Approved by: https://github.com/justinchuby
2023-03-09 00:00:44 +00:00
0f4652f498 [ONNX] Merge 'initializers' into 'TorchScriptGraph' (#95676)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/95676
Approved by: https://github.com/titaiwangms, https://github.com/wschin
2023-03-08 21:12:20 +00:00
28e69954a1 [ONNX] Support aten::bit_wise_not in fx-onnx exporter (#94919)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/94919
Approved by: https://github.com/justinchuby, https://github.com/wschin
2023-02-16 06:21:59 +00:00
055dc72dba [ONNX] Bump onnx to 1.13.1, onnxruntime to 1.14.0 (#94767)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/94767
Approved by: https://github.com/abock
2023-02-14 03:53:05 +00:00
a27bd42bb9 [ONNX] Use onnxruntime to run fx tests (#94638)
- Enable the mnist test
- Removed `max_pool2d` in the test because we don't have the op yet.
- Add aten::convolution
- Bump onnxscript version
Pull Request resolved: https://github.com/pytorch/pytorch/pull/94638
Approved by: https://github.com/BowenBao, https://github.com/wschin, https://github.com/titaiwangms
2023-02-11 15:32:03 +00:00
88d0235b73 [ONNX] Update CI test environment; Add symbolic functions (#94564)
* CI Test environment to install onnx and onnx-script.
* Add symbolic function for `bitwise_or`, `convert_element_type` and `masked_fill_`.
* Update symbolic function for `slice` and `arange`.
* Update .pyi signature for `_jit_pass_onnx_graph_shape_type_inference`.

Co-authored-by: Wei-Sheng Chin <wschin@outlook.com>
Co-authored-by: Ti-Tai Wang <titaiwang@microsoft.com>
Pull Request resolved: https://github.com/pytorch/pytorch/pull/94564
Approved by: https://github.com/abock
2023-02-10 20:44:59 +00:00
6d722dba0f [ONNX] Update CI onnx and ORT version (#94439)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/94439
Approved by: https://github.com/BowenBao
2023-02-09 04:08:38 +00:00
b453adc945 [BE][CI] rename .jenkins (#92845)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/92845
Approved by: https://github.com/clee2000
2023-01-25 23:47:38 +00:00
afe6ea884f Revert "[BE][CI] rename .jenkins to .ci, add symlink (#92621)"
This reverts commit 8972a9fe6aa8be8f8035c83094ed371973bfbe73.

Reverted https://github.com/pytorch/pytorch/pull/92621 on behalf of https://github.com/atalman due to breaks shipit
2023-01-23 15:04:58 +00:00
b5f614c4cd Move ASAN and ONNX to Python 3.9 and 3.8 (#92712)
As 3.7 is getting deprecated
Pull Request resolved: https://github.com/pytorch/pytorch/pull/92712
Approved by: https://github.com/weiwangmeta, https://github.com/kit1980, https://github.com/seemethere
2023-01-23 14:46:02 +00:00
8972a9fe6a [BE][CI] rename .jenkins to .ci, add symlink (#92621)
Pull Request resolved: https://github.com/pytorch/pytorch/pull/92621
Approved by: https://github.com/huydhn, https://github.com/ZainRizvi
2023-01-21 02:40:18 +00:00