ban kwargs?

[MatthewCaseres]

I don't think the model you developed supports them. they are pretty annoying, should I stop supporting them as well? Should you start supporting them? Does it matter?

[lewisfogden]

hmm I use them a lot, e.g. passing in the data/basis and any top level functions, e.g.:

Is that where you are meaning?

[MatthewCaseres]

it appears that your code does not recognize kwargs as a key in the cache

https://github.com/lewisfogden/heavylight/blob/main/src/heavylight/heavylight.py#L17

[lewisfogden]

Oh I see! Yes I think pretty much all methods are positional (and 99% are t), so kwargs won't work.

Would we just need to update to

def __call__(self, *arg, /):

Also tempted just to leave it as it'll warn people anyway, and pop something in the docs.