coverage: 99.719%. remained the same when pullinge1b00d6 on Transurgeon:adding-matrix-order into45ccf4b on python-graphblas:main.

Thanks for starting on this@Transurgeon!

I would recommend adding implementation too. Ideally, you should test and verify what you did is needed and works as expected as you addorder= arguments. Perhaps this also means addingmy_matrix.orientation attribute too, or just using the existingmy_matrix.ss.orientation attribute.

I would start withA.T.new(order="columnwise") and test that it works (add tests at the end ofgraphblas/tests/test_matrix.py).

Also,graphblas/core/utils.py:get_order will be helpful.

Hello@eriknw , sorry for the delay on this, I have been finishing up some details for my GSoC project.
I tried to implement the orientation today, however, it is working as planned.
I have added the argument to bothMatrix.__new__ andTransposeMatrix.new(). I also calledget_order like you recommended, but the test that I have added doesn't seem to pass like I would expect.
I am starting to get lost in the layers of OOP abstraction.
More importantly, I am also having problems when debugging due to the error below, do you have any ideas on how to fix this?

P.S Is the expectation to addproperty.getter/setter for.ss.orientation? I thought that could be a solution but wasn't sure if it was what you wanted.

No worries :) I was also traveling and mostly AFK last week.

I will take a closer look at this PR. As for the error, it looks like you're running from within thepython-graphblas/graphblas directory (a builtin library is trying toimport select, and it's picking upgraphblas/select, which breaks). Try running frompython-graphblas directory.

Hey, sorry for letting this sit. I was traveling and on vacation the last two weeks. I should have time to give this attention later this week.