Nic Washbourne
Nic Washbourne
Yeah my plan was to write some more extensive tests for complex (since the existing ones didn't catch all of the break cases I've found) and add in a few...
> Isn't this an unbounded cache? @FabioLuporini technically unbounded yes, but entries are evicted by `WeakValueDictionary` when the referred object stops existing, so cache entries are only stored while that...
> How much memory growth have you observed with a big Operator (ideally something high order)? On that demo script from you I've been using, memory usage was within the...
> does it make sense to add a test that ensures we're not leaking memory and/or memory consumption is bounded ? this is mostly me being paranoid @FabioLuporini the memory...
Yeah will cook up a demo. Did run the test suite yesterday but some of my typing changes for the review must have broken things, will take a look
Profiled with [this script](https://github.com/devitocodes/devito/blob/548d5a4f54733f20621b5877c228199d9fc3937d/tests/demo.py) on dewback through numactl, getting a less substantial but measurable 5% improvement overall: you can view the cProfile output from both [the upstream head](https://gist.github.com/enwask/da23dbb6978e59add1120c91c725c6c8#file-prof_old_visitors-txt) and my...
Noticed that the `Search` rewrite is actually a regression; trying to work out why... Found out why! Turns out `bfs_first_hit` is neither a BFS nor does it return only the...
> It would probably be worth adding a run to CI-core which runs the tests with free-threaded Python Thought about this but I think lots of things will fail off...
> That's a reasonable contention, but they will all need fixing eventually. We've been tripped up like this before (CXX anyone?) Yeah, just wasn't sure that we want CI to...