danielward27 / flowjax

https://danielward27.github.io/flowjax/
MIT License
82 stars 10 forks source link

Fix param, static order in MaximumLikelihoodLoss #148

Closed danielward27 closed 5 months ago

danielward27 commented 5 months ago

Using eqx.combine with static first causes problems with NonTrainable, as we get a None in params, which causes the structures to not match when combining. Briefly, this is because eqx.combine(None, [1,1]) works, and eqx.combine([1,1], None) is not allowed as the "structures do not match".