This reverts commit dea23730.

This avoids quadratic runtime in the previous "batchy toposort".

Co-authored-by: Andreas Kloeckner <inform@tiker.net>

Matt Smith authored 1 year ago and Andreas Klöckner committed 1 year ago

"__call__ = rec" is wrong, doesn't work if subclass overrides rec

* replace immutables.Map with immutabledict

* Use Mapping rather than any concrete type in public type annotations

---------

Co-authored-by: Andreas Kloeckner <inform@tiker.net>

Andreas Klöckner authored 1 year ago and Andreas Klöckner committed 1 year ago

Also use cache_hash from attrs instead of custom cached implementation

Andreas Klöckner authored 1 year ago and Andreas Klöckner committed 1 year ago

Unlike BoundPyOpenCLProgram, this is specific to a CL context, and
thus can cache more effectively, avoid holding long-lived references
to CL contexts in non-context-specific settings, and use
loopy.ExecutorBase, towards some of the same goals.

* Disable bounds checking when no loopy call present

* add comment

Andreas Klöckner authored 1 year ago and Andreas Klöckner committed 1 year ago

Also, use distributed vis functionality even in the non-distributed
case, to cut down on code duplication.

Kaushik Kulkarni authored 2 years ago and Andreas Klöckner committed 1 year ago

Co-authored-by: Andreas Kloeckner <andreask@illinois.edu>