Summary:
all jobs are currently running with "--dry-run", so you can verify if the jobs are doing the right thing. i'll remove the flag and make it runs every hour same as on Jenkins once this PR is approved.
Pull Request resolved: https://github.com/pytorch/pytorch/pull/30996
Differential Revision: D18971001
Pulled By: mingbowan
fbshipit-source-id: 2384bdb50ebdf47aad265395f26be3843f0ce05e
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/31164
We have a small number of internal projects that still are on Python 2.
Until we can figure out how to get rid of them, we need to continue
supporting Python 2 for PyTorch.
Test Plan: Imported from OSS
Differential Revision: D18949698
Pulled By: suo
fbshipit-source-id: 4a9d7e4306ed81576e05f243de472937a2bb1176
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/31100
This appears to not work right now. Disabling pending an investigation.
Test Plan: Imported from OSS
Differential Revision: D18928777
Pulled By: suo
fbshipit-source-id: 63089131bad98902979e5cf4373732c85badef9d
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/29357
As title
Test Plan: Imported from OSS
Reviewed By: pritamdamania87
Differential Revision: D18920562
Pulled By: suo
fbshipit-source-id: b5dd559cfb0ba6c64b9ccf3655417afb56a7b472
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/30892
Fixes all outstanding lints and actually installs a properly configured
flake8
Test Plan: Imported from OSS
Differential Revision: D18862825
Pulled By: suo
fbshipit-source-id: 08e9083338a7309272e17bb803feaa42e348aa85
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/28933
Merge all the things that don't add annotations into a single
`quick-checks` job. This helps reduce concurrency and clutter
at the top of the status check page.
This doesn't touch the actually important items (flake8 + clang-tidy),
but those are a little trickier to handle because of how annotations are
added.
Test Plan: Imported from OSS
Differential Revision: D18235396
Pulled By: suo
fbshipit-source-id: 8fba44f3f5d398b1dce0f39f51d6652f3e0c1bf7
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/28879
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Test Plan: Imported from OSS
Differential Revision: D18231741
Pulled By: ezyang
fbshipit-source-id: d49711ad41d7ff7e527326c68fd8db86da10a818
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/28842
We don't care which python version, and github actions has changed the
versions available, breaking our CI. So just pin it to 3-something to
make it more future proof
Test Plan: Imported from OSS
Differential Revision: D18205349
Pulled By: suo
fbshipit-source-id: bf260dc29a138dd8bf8c85081a182aae298fe86d
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/28788
Okay, my last fix was wrong because it turns out that the base SHA is
computed at PR time using the actual repo's view of the base ref, not
the user's. So if the user doesn't rebase on top of the latest master
before putting up the PR, the diff thing is wrong anyway.
This PR fixes the issue by not relying on any of these API details and
just getting the merge-base of the base and head refs, which should
guarantee we are diffing against the right thing.
This solution taken from https://github.com/github/VisualStudio/pull/1008
Test Plan: Imported from OSS
Differential Revision: D18172391
Pulled By: suo
fbshipit-source-id: 491a50119194508b2eefa5bd39fe813ca85f27b1
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/28388
The clang-tidy script diffs the PR head ref against the base ref so that
it works only on changed lines. If the base ref is a stale `master`,
then the script will fetch upstream `master` and potentially report
unrelated changes in the diff
Use the base sha instead of ref so that the revision that the script
diffs against is stable.
Test Plan: Imported from OSS
Differential Revision: D18051363
Pulled By: suo
fbshipit-source-id: 80ead2f837e2d6244245ed7b576e84a99f0ea035
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/27805
The expressions syntax for actions is pretty cool! Using it to clean up
some of my convoluted checks from before
Test Plan: Imported from OSS
Differential Revision: D17909353
Pulled By: suo
fbshipit-source-id: 8b9a85476ba19452f48c532a2daed830f074088a
Summary:
whoops, this got left in by accident
Pull Request resolved: https://github.com/pytorch/pytorch/pull/27796
Differential Revision: D17892482
Pulled By: suo
fbshipit-source-id: f92255d78fe70d3c22c4422b6333ac288cb330d6
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/27755
This gives us nice annotations. See
https://github.com/suo/pytorch/pull/22/files for an approximation of
what it will look like (ignore the warnings on the lint.yml file).
I deleted the old azure pipelines one since making the code work for
both was annoying, and unlike flake8 this one does not affect master
Test Plan: Imported from OSS
Differential Revision: D17888974
Pulled By: suo
fbshipit-source-id: d8928a1451b6ef500dc1889284cab2845ecdeeea
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/26694
Previously we would not properly populate `errorDesc` for:
```
./torch/jit/__init__.py:13:1: F401 'torch.nn.ModuleList' imported but unused
```
because we wanted only letters and spaces. Be more permissive
Test Plan: Imported from OSS
Differential Revision: D17551999
Pulled By: suo
fbshipit-source-id: b82567df1fa3c9729e7427dc3461bedfb40933dc
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/26562
I was trying to be too clever with GITHUB_HEAD_REF...
Test Plan: Imported from OSS
Reviewed By: driazati
Differential Revision: D17538517
Pulled By: suo
fbshipit-source-id: 82c71ee3c6edb299ac8eb73675d96967e00a29f1
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/25824
Use github actions for flake8. This is nice because it makes it easier
to create inline annotations for lint violations.
It ends up looking like this:
https://github.com/suo/pytorch/pull/21/files
Test Plan: Imported from OSS
Differential Revision: D17487007
Pulled By: suo
fbshipit-source-id: 663094ea2bbbdb1da5b7e5d294c70735a319d5e5
* Add environment collection script
Fixes#6111. This should make it easier for users to report bugs by giving
them a script to collect system environment information.
Changes include:
- Refactor out the environment collecting code from utils.bottleneck
- Add script (collect_env.py)
- Cleaned up the issues template so that it suggests using the script
and is more readable.
Testing: added expect tests to go with 4 CI configurations. Whenever one
of these configurations gets updated, the test will fail until the test
also gets updated.
* Expect tests
* Update issue template
* Fix random space
* Minor improvement to issue template; fix expect test
* Skip expect test if BUILD_ENVIRONMENT not found; test fix; split off smoke/expect test
