search

inducer / pyopencl

OpenCL integration for Python, plus shiny features
http://mathema.tician.de/software/pyopencl
Other
1.05k stars 240 forks source link

Use dataclass instead of pytools.Record #571

Closed alexfikl closed 2 years ago

alexfikl commented 2 years ago

The change in reduction.py has some actual logic changes (made _ReductionInfo frozen so had to move things around), but the rest should be cleanup.

Most of these classes seemed internal, so I assume they don't need some compatibility with Record (?).

inducer commented 2 years ago

Thanks! I'm open to a mutually dependent change in boxtree to help us get rid of Record.

alexfikl commented 2 years ago

Thanks! I'm open to a mutually dependent change in boxtree to help us get rid of Record.

I think I just missed a member in BuiltList, so hopefully the latest version will fix boxtree.

inducer commented 2 years ago

Thx!