mirror of
https://github.com/tinygrad/tinygrad.git
synced 2026-06-24 02:14:17 +00:00
hotfix: nobody uses [run_process_replay] [pr] (#7264)
This commit is contained in:
parent
fa5dc7857a
commit
d482d927a8
3 changed files with 3 additions and 4 deletions
|
|
@ -178,4 +178,4 @@ python3 -m pytest test/ # whole test suite
|
|||
|
||||
#### Process replay tests
|
||||
|
||||
[Process replay](https://github.com/tinygrad/tinygrad/blob/master/test/external/process_replay/process_replay.py) compares your PR's generated kernels against master. If your PR is a refactor or speedup without any expected behavior change, It should include [pr] in the pull request title, [example](https://github.com/tinygrad/tinygrad/pull/4995). Note that you should keep your branch up-to-date with master.
|
||||
[Process replay](https://github.com/tinygrad/tinygrad/blob/master/test/external/process_replay/README.md) compares your PR's generated kernels against master. If your PR is a refactor or speedup without any expected behavior change, It should include [pr] in the pull request title.
|
||||
|
|
|
|||
2
test/external/process_replay/README.md
vendored
2
test/external/process_replay/README.md
vendored
|
|
@ -2,7 +2,7 @@
|
|||
|
||||
Process replay is a tool for creating a diff of generated kernels between two commits. By default, process replay doesn't assert kernel diffs.
|
||||
|
||||
Refactor and speedup prs must enable the assert by including `[pr]` in the pull request title.
|
||||
Refactor and speedup PRs must enable the assert by including `[pr]` in the pull request title.
|
||||
|
||||
Note that process replay [early stops when over 20% of kernels change, for speed.](https://github.com/tinygrad/tinygrad/pull/5480).
|
||||
|
||||
|
|
|
|||
|
|
@ -22,8 +22,7 @@ early_stop = multiprocessing.Event()
|
|||
logging.basicConfig(level=logging.INFO, format="%(message)s")
|
||||
|
||||
# user config
|
||||
ASSERT_FLAGS = {"[pr]", "[run_process_replay]"}
|
||||
ASSERT_DIFF = int(any(flag in os.getenv("COMMIT_MESSAGE", flag) or flag in os.getenv("PR_TITLE", flag) for flag in ASSERT_FLAGS))
|
||||
ASSERT_DIFF = int((flag:="[pr]") in os.getenv("COMMIT_MESSAGE", flag) or flag in os.getenv("PR_TITLE", flag))
|
||||
if not getenv("ASSERT_PROCESS_REPLAY", 1): ASSERT_DIFF = 0
|
||||
SKIP_PROCESS_REPLAY = (k:="[skip_process_replay]") in os.getenv("COMMIT_MESSAGE", "") or k in os.getenv("PR_TITLE", "")
|
||||
if REF == "master": SKIP_PROCESS_REPLAY = True
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue