Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR to run CI pipeline #893

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions test/line_attribution_tests/line_after_final_alloc.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@

def main():
accum = bytes()
for i in range(31):
accum += bytes(10485767 * 2)

print(len(accum))

asdf = bytes(2 * 10485767)

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable asdf is not used.
some_dead_line = None

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable some_dead_line is not used.


if __name__ == '__main__':
main()
13 changes: 13 additions & 0 deletions test/line_attribution_tests/loop_below_threshold.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@

def main():
accum = bytes()
for i in range(31):
accum += bytes(10485767 // 4) # far below the allocation sampling window

print(len(accum))

asdf = bytes(2 * 10485767)

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable asdf is not used.


if __name__ == '__main__':
main()
13 changes: 13 additions & 0 deletions test/line_attribution_tests/loop_with_multiple_lines.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@

def main():
accum = bytes()
for i in range(31):
accum += bytes(2 * 10485767) # 2x the allocation sampling window
bogus = None

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable bogus is not used.
print(len(accum))

asdf = bytes(2 * 10485767)

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable asdf is not used.


if __name__ == '__main__':
main()
13 changes: 13 additions & 0 deletions test/line_attribution_tests/loop_with_one_alloc.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@

def main():
accum = bytes()
for i in range(31):
accum += bytes(2 * 10485767) # 2x the allocation sampling window

print(len(accum))

asdf = bytes(2 * 10485767)

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable asdf is not used.


if __name__ == '__main__':
main()
12 changes: 12 additions & 0 deletions test/line_attribution_tests/loop_with_two_allocs.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@

def main():
accum = bytes()
for i in range(31):
accum += bytes(2 * 10485767) + bytes(2 * 10485767) # 2x the allocation sampling window

print(len(accum))

asdf = bytes(2 * 10485767)

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable asdf is not used.

if __name__ == '__main__':
main()
111 changes: 111 additions & 0 deletions test/smoketest_line_invalidation.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,111 @@
"""
This is bound very closely to the current implementation of
the tests in `test/line_attribution_tests.

The two things that matter are the number of loops, the size
of the allocations, and the exact line numbers.


"""

expected_md5_sums = {
"line_attribution_tests/loop_below_threshold.py": "7664a7dcc0f4ab94a44e431448b5f348",
"line_attribution_tests/loop_with_one_alloc.py": "da9ff0aa223123c956049e940c3ef093",
"line_attribution_tests/loop_with_multiple_lines.py": "48ce0e8693fe43b1ebb7eb75a0fd5832",
"line_attribution_tests/loop_with_two_allocs.py": "71f337140aa25383525e56a6167cabf8",
"line_attribution_tests/line_after_final_alloc.py": "ca8cdd44ea6e4a9c286c05facae6a721"
}

import subprocess
import tempfile
import sys
from typing import List

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'List' is not used.
from pathlib import Path
from hashlib import md5
from scalene.scalene_json import ScaleneJSONSchema

N_LOOPS = 31
LOOP_ALLOC_LINENO = 5 #
OUT_OF_LOOP_ALLOC_LINENO = 9

def check_for_changes():
errors = []
for fname, expected_sum in expected_md5_sums.items():
with open(fname, 'rb') as f:
digest = md5(f.read()).hexdigest()
if digest != expected_sum:
errors.append(fname)
assert len(errors) == 0, f'Detected change in file(s) {",".join(errors)}'

def get_line(scalene_profile: ScaleneJSONSchema, lineno: int):
files = list(scalene_profile.files.keys())
assert len(files) == 1
filename = files[0]
return scalene_profile.files[filename].lines[lineno - 1]




def get_profile(test_stem, outdir_p, test_dir):
proc = subprocess.run(

Check notice

Code scanning / CodeQL

Unused local variable Note test

Variable proc is not used.
[
sys.executable,
"-m",
"scalene",
"--cli",
"--json",
"--outfile",
outdir_p / f"{test_stem}.json",
test_dir / f"{test_stem}.py",
],
# capture_output=True,
check=True,
)
with open(outdir_p / f"{test_stem}.json", "r") as f:
profile = ScaleneJSONSchema.model_validate_json(f.read())
return (test_stem, profile)


def main():
test_dir = Path(__file__).parent / "line_attribution_tests"
with tempfile.TemporaryDirectory() as outdir:
outdir_p = Path(outdir)
one_alloc = get_profile("loop_with_one_alloc", outdir_p, test_dir)
two_on_one_line = get_profile("loop_with_two_allocs", outdir_p, test_dir)
below_threshold = get_profile("loop_below_threshold", outdir_p, test_dir)
line_after_final_alloc = get_profile(
"line_after_final_alloc", outdir_p, test_dir
)
errors = []
for stem, profile in [one_alloc, two_on_one_line, below_threshold, line_after_final_alloc]:
assert isinstance(profile, ScaleneJSONSchema)
if not any(line.n_malloc_mb > 0 for line in profile.files[list(profile.files.keys())[0]].lines):
errors.append(f"Failed to find any memory allocated in {stem}")
# for stem, profile in [one_alloc, two_on_one_line, line_after_final_alloc]:
# line = get_line(profile, LOOP_ALLOC_LINENO)
# if not line.n_mallocs == N_LOOPS:
# errors.append(f"Expected {N_LOOPS} distinct lines on {stem}, got {line.n_mallocs} on line {LOOP_ALLOC_LINENO}")
Comment on lines +84 to +87

Check notice

Code scanning / CodeQL

Commented-out code Note test

This comment appears to contain commented-out code.

# bt_stem, bt_prof = below_threshold
# bt_line = get_line(bt_prof, LOOP_ALLOC_LINENO)
# if not bt_line.n_mallocs < N_LOOPS:
# errors.append(f"{bt_stem} makes smaller allocations than the allocation sampling window, so fewer than {N_LOOPS} allocations on {LOOP_ALLOC_LINENO} should be reported. Got {bt_line.n_mallocs} mallocs")

# for stem, profile in [one_alloc, two_on_one_line, below_threshold, line_after_final_alloc]:
# line = get_line(profile, OUT_OF_LOOP_ALLOC_LINENO)
# if not line.n_mallocs == 1:
# errors.append(f'Line {OUT_OF_LOOP_ALLOC_LINENO} in {stem} makes a large allocation, so it should be reported.')
Comment on lines +91 to +97

Check notice

Code scanning / CodeQL

Commented-out code Note test

This comment appears to contain commented-out code.

if len(errors) > 0:
for error in errors:
print(f'ERROR: {error}')
for profile in [one_alloc, two_on_one_line, below_threshold, line_after_final_alloc]:
print("\n\n\n\n")
print(profile[1].model_dump_json(indent=4))
exit(1)
else:
print("PASS")
exit(0)

if __name__ == '__main__':
main()
Loading