-
Notifications
You must be signed in to change notification settings - Fork 0
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
Automatically collect call graph from tdd_repro
#15
Automatically collect call graph from tdd_repro
#15
Conversation
…-864-6a-test-initial-runtime-data-hypotheses
c192533
to
cd9f723
Compare
…-864-6a-test-initial-runtime-data-hypotheses
tdd_repro
@@ -0,0 +1,2272 @@ | |||
2024-10-01 11:28:38,309 WARNING ⚠ TDD is enabled. Agents have access to the test patch and will not try to reproduce the problem. ⚠ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we remove these handpicked logs? These are really large and don't seem necessary without associated docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like the repo is the perfect place to keep track of large log files that sample our investigations.
Where do you suggest keeping them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, to rephrase, what are these log files used for? If we're just storing these log files here we could use google drive instead.
sweagent/environment/swe_env.py
Outdated
# Inject test injecter: | ||
# TODO: generalize the file location | ||
# (Choosing urls.py is a hack: we know that `urls.py` is one of the dynamic imports from runtests.py, so appending to it works.) | ||
test_file_to_override = os.path.join(container_test_folder, "urls.py") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: I still need to fix this before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabled for now
…arted running circles multiple times.
…-864-6a-test-initial-runtime-data-hypotheses
tdd_repro
command, with some customizations.django__django-11299
(PRO-864).