Photonic-Architecture-Laboratories / si-photonics-toolkit

Photonic Integrated Design Package
MIT License
11 stars 0 forks source link

Requirements #25

Closed esmagden closed 1 year ago

esmagden commented 1 year ago

Jax is required for map_coordinates, but is trax also in fact a requirement?

It feels like it shouldn't be, and we should remove it if that's the case

aycandv commented 1 year ago

Trax is a requirement because trax.fastmath.grad is used in effective_index module.

esmagden commented 1 year ago

trax was removed