Christian Balcom
Christian Balcom
The absolute path changes have been removed, instead fetching the current sandbox path when calculating paths for fileio actions. I also added a "SANDBOX_TIMEOUT" configuration that is shared with the...
Looks like the integration test failure is due to the prompt messages changing.
It was definitely a pain to diagnose and fix as-is. I had to get the tests running locally with a debugger before I realized what was going wrong, and I...
It was passing the integration tests when I last committed, looks like your merge is what broke it.
I also don't think I can commit to this branch, so you will have to make the fixes unless I re-create the original PR from my branch.
I have resolved the issue in a new pr from my branch, which is now ready for review https://github.com/OpenDevin/OpenDevin/pull/1417
@rbren should be ready to go
This integration test failure does not occur locally for some reason, investigating.
@rbren I have fixed the integration test issue. It was introduced when the local-sandbox integration test was added late last night https://github.com/OpenDevin/OpenDevin/commit/7d5856e36b69eb31740117b62e2f88d6abe343f6 . The test environment configuration did not account...
I ran into this recently with bf16 tensors in my own onnx runtime. Pytorch automatically upcasts to fp32 on a number of ops, and any models trained to that standard...