search

inducer / arraycontext

Choose your favorite numpy-workalike!
6 stars 11 forks source link

Extend pytest_generate_tests_for #44

Closed alexfikl closed 3 years ago

alexfikl commented 3 years ago

This is built on top of #43, so it has a few annoying commits in there (see alexfikl/arraycontext#1 for a nicer full diff). Only a305c3c and 32be014 should be in here.

It adds a pytest_generate_tests_for_array_context that takes an impls argument, similar to what was proposed in #14. It should be easy to add other array contexts as well, so hopefully it can be used for the lazy one too.

inducer commented 3 years ago

Thanks! Left a review at https://github.com/alexfikl/arraycontext/pull/1#pullrequestreview-691195063.

alexfikl commented 3 years ago

I'll go ahead and merge this back into #43 due to https://github.com/inducer/arraycontext/pull/43#discussion_r657496291. All the comments in alexfikl/arraycontext#1 should be addressed (or at least ready for another review).