lava icon indicating copy to clipboard operation
lava copied to clipboard

Incorrect use of np.logical_and/or in Connection ProcessModels

Open gkarray opened this issue 3 years ago • 0 comments

Objective of issue: Incorrect use of np.logical_and and np.logical_or was discovered in learning-related code in Connection ProcessModels.

Lava version:

  • [ ] 0.6.0 (feature release)
  • [ ] 0.5.1 (bug fixes)
  • [x] 0.5.0 (current version)
  • [ ] 0.4.1
  • [ ] 0.4.0
  • [ ] 0.3.0
  • [ ] 0.1.2

I'm submitting a ...

  • [x] bug report
  • [ ] feature request
  • [ ] documentation request

Current behavior:

  • Currently, to implement and and or element-wise operations on 3 np.ndarray (example : a, b and c), the following is used : np.logical_and(a, b, c)

Expected behavior:

  • It should be implemented like this : np.logical_and(np.logical_and(a, b), c) or a & b & c

gkarray avatar Oct 13 '22 11:10 gkarray