QuantEcon.py icon indicating copy to clipboard operation
QuantEcon.py copied to clipboard

Migrate `np.dot` to use python @

Open mmcky opened this issue 4 years ago • 3 comments

It would be nice to migrate np.dot to use the inbuilt python @ operator for matrix multiplication.

Some thought is required as np.dot doesn't behave in the exact same way as @

https://stackoverflow.com/questions/34142485/difference-between-numpy-dot-and-python-3-5-matrix-multiplication

mmcky avatar Nov 11 '21 00:11 mmcky

Hi! I've started to work on this change (https://github.com/nshea3/QuantEcon.py/tree/migrate-matmul-symbol)

Changes to lqnash and lqcontrol are both in progress. np.dot is used in many places in the repo so I think it will be a few weeks before I send along a PR.

Thanks for the stackoverflow post, useful info there!

Nick

nshea3 avatar Nov 17 '21 05:11 nshea3

thanks @nshea3 -- appreciate it.

mmcky avatar Nov 18 '21 05:11 mmcky

Hi everyone, I will take this issue. Let me know if there is anyone else working on this :)

HumphreyYang avatar Jan 03 '23 23:01 HumphreyYang