Conversation
|
That diff is genuinely unhelpful, but the end result looks OK. As for your |
|
Thanks! |
I mostly copied that from some stuff I managed to sneak by you a long time ago in sumpy. What would be a better way to optimize those kernels a bit? |
|
Got a whole course on that: https://relate.cs.illinois.edu/course/cs598apk-f18/ :) But I'd be happy to chat on Thursday. Which ones are the expensive ones? |
It was just meant to do a bare minimum sort of thing, since I haven't profiled anything. For the moment, all the geometries we've been playing with have like 2-3000 points, so it probably doesn't matter that much anyway. Thanks! I'll give that a read! |
This takes out another chunk out of #30 that just changes the proxy generator. The bigger changes are
ProxyGenerator, that ignores any QBX expansions, and aQBXProxyGenerator, that makes proxies outside of the QBX expansions.gather_block_interaction_points, which was meant to just make a big array of neighbors + proxies to be evaluated together.Looking at the diff, it looks a bit big, but it should just be moving things around.