ACEsuit / Polynomials4ML.jl

Polynomials for ML: fast evaluation, batching, differentiation
MIT License
12 stars 5 forks source link

Fix release input in `PooledSparseProduct` and tuple inputs #79

Closed CheukHinHoJerry closed 8 months ago

CheukHinHoJerry commented 9 months ago

Continuing from discussion 1-2 weeks ago on the extra allocations in evaluation of spherical harmonics observed in EQM. The reason is that PooledSparseProductLayer did not release layer properly.

Major updates from this PR:

Minor fix:

I also ran the test of EQM locally and it works fine.

cortner commented 8 months ago

can I review and merge?