Jan Vesely

Results 130 comments of Jan Vesely

can you add: ``` Fixes: https://github.com/PrincetonUniversity/PsyNeuLink/issues/1703 ``` to merge commit description? it will help with issue tracking. another nit: I'd be nice to remove the 33 newly introduced redundant imports...

looks like the bug existed before 30f60b2. I can still see it after reverting said commit. It looks to be tied to the original training in `get_trained_network_multLCA` not using compiled...

sorry for the delay. The issue is still reproducible on devel (8bf0f7ff932bd5b1cb0caef52bbd6175ba357b04) Here's a reproducer: `MNET=Python python Scripts/Debug/Yotam\ LCA\ Model\ LLVM.py` The patch from comment https://github.com/PrincetonUniversity/PsyNeuLink/issues/1808#issuecomment-718244998 no longer applies.

Moreover the current constraint is

Tested on the fit2 branch. @dependabot squash and merge