Commit Graph

31 Commits

Author SHA1 Message Date
Sergii Tkachenko de6ed9ba9f
[Python] Migrate from yapf to black (#33138)
- Switched  from yapf to black
- Reconfigure isort for black
- Resolve black/pylint idiosyncrasies 

Note: I used `--experimental-string-processing` because black was
producing "implicit string concatenation", similar to what described
here: https://github.com/psf/black/issues/1837. While currently this
feature is experimental, it will be enabled by default:
https://github.com/psf/black/issues/2188. After running black with the
new string processing so that the generated code merges these `"hello" "
world"` strings concatenations, then I removed
`--experimental-string-processing` for stability, and regenerated the
code again.

To the reviewer: don't even try to open "Files Changed" tab 😄 It's
better to review commit-by-commit, and ignore `run black and isort`.
2023-06-09 15:08:55 -07:00
Craig Tiller 1f1f923a72
[benchmarks] Remove stats integration (#30900)
* remove old stats cruft

* remove

* remove

* fix

* fix
2022-09-09 04:15:12 -07:00
AJ Heller 3c558dd796
Disable performance labeler for Github PRs (#29258) 2022-03-29 19:51:33 -07:00
Craig Tiller 6003710534
Label microbenchmark differences similarly to bloat (#27998)
* benchmark differences as a label

* debug

* Automated change: Fix sanity tests

Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2021-11-11 07:37:02 -08:00
Richard Belleville b6cc72f129
Revert "Run 2to3 on tools directory (#26002)" (#27183)
This reverts commit a388361c3d.
2021-08-30 12:29:20 -07:00
Richard Belleville a388361c3d
Run 2to3 on tools directory (#26002)
* Run 2to3 on tools directory

* Delete github_stats_tracking

* Re-run 2to3

* Remove unused script

* Remove unused script

* Remove unused line count utility

* Yapf. Isort

* Remove accidentally included file

* Restore line_count directory
2021-08-30 09:25:21 -07:00
AJ Heller 5139a012e7
Migrate the bm_diff benchmarks to python3 (#25619)
* Migrate the bm_diff benchmarks to python3

Includes a requirements.txt pinned at ~2017 versions, when this script
was first written.

* Replace p2 with p3 scipy/numpy dependencies.

* py2->3 for benchmark setup scripts

* upgrade pip to resolve python3 cryptography/setuptools-rust problem

* re-add jobset import (accidentally removed)

* re-add six's urllib import. This file is still used in py2 tests

* force py3 in run_if_c_cpp_modified.sh

* Fix another instance of subprocess.check_output binary mode

* Use the requirements.txt for CI perf environment setup

* Try to upgrade PyJWT. (v2.0.0 was problematic, #25053)

v2.x makes encode return strs from jwt.encode in both py2 and py3.
Previously, py3 would return bytes, and py2 a str.

* upgate cryptography lib version requirements for jwt.

* Wrap pip requirements specifier in quotes '>=x,<y'

* Decode subprocess output once instead of for every line

* Revert "Decode subprocess output once instead of for every line"

This reverts commit 28d14026431622ac7afd3535a8d7118e5be96628.

py2 doesn't support the `text` argument to subprocess.check_output.

* Address reviewer requests

* Pin a valid scipy version

* Remove scipy and tabulate dependencies from macos tests
2021-03-09 15:44:00 +01:00
Esun Kim 7be821ac47 Yapf all target python sources 2020-01-03 09:25:10 -08:00
apolcyn 8a2c9512a5
Merge pull request #20447 from apolcyn/fix_print
Put parenthesis around print in bm_main.py
2019-10-02 16:20:54 -07:00
Alexander Polcyn 76abd3dd03 Also fix exception syntax 2019-10-02 13:12:06 -07:00
Alexander Polcyn c6654fc02a Put parenthesis around print in bm_main.py 2019-10-02 12:06:03 -07:00
chentanjun 4ecc1fe6a4 fix the wrong word 2019-10-01 05:13:33 +08:00
Lidi Zheng 0f0822d53f WIP: Utilize the GitHub Check Feature 2018-12-13 16:40:47 -08:00
Mehrdad Afshari 87cd994b04 Upgrade yapf to 0.20.0
Upgrade yapf version to 0.20.0 and reformat Python files.
2018-01-02 14:40:00 -08:00
ncteisen 173c477bd0 tools/profiling 2017-12-11 16:52:44 -08:00
ncteisen 4df6a64403 Change default bm_diff loops to 10 2017-11-06 15:46:37 -08:00
ncteisen 460af54440 Shuffle new and old jobs together 2017-09-28 16:12:15 -07:00
ncteisen 5c92dcde1e Allow passing regex to bm_diff tools 2017-07-20 14:55:49 -07:00
ncteisen b423f579e4 Disambiguate trickle and normal bms 2017-06-14 11:55:40 -07:00
ncteisen 6dc4c7f5d9 Disable counters for trickle, inc timeout 2017-06-14 11:07:52 -07:00
ncteisen 6a5e923a81 Only output crashes if they occur 2017-06-08 23:29:02 -07:00
ncteisen 0ff7384f6d Merge branch 'master' of https://github.com/grpc/grpc into faster-bm-diff 2017-06-08 08:15:03 -07:00
ncteisen 07639167fc 2 space indentation 2017-06-08 08:08:25 -07:00
ncteisen 0ac47d28cd Github comments 2017-06-07 15:19:24 -07:00
ncteisen 7cd7b7fc43 Differentiate between timeouts and crashes 2017-06-07 15:14:19 -07:00
ncteisen 2b96949d3d Yapf fmt 2017-06-01 21:59:46 -07:00
ncteisen a278759ec9 Add flakiness data to PR comment 2017-06-01 15:32:08 -07:00
ncteisen d21040bf1e Update readme 2017-05-12 00:46:43 -07:00
ncteisen 738be24db4 Py fmt 2017-05-10 16:22:54 -07:00
ncteisen 48d973a276 Add readme and flags 2017-05-10 16:22:54 -07:00
ncteisen 64637b7c8c restructure 2017-05-10 16:22:43 -07:00