On Mon, 22 May 2017, Vincent Delecroix wrote:

1) add an option inplace=True/False. There is at least one such method in Sage: the relabel method for graphs. The behavior is as follows:

inplace is mentioned at http://doc.sagemath.org/html/en/developer/coding_basics.html#miscellanous-minor-things

So yes, I think "inplace=True" should be preferred over "return_copy=False" or similar.

2) Make a private inplace method mat._transpose_inplace()

3) Make a public inplace method mat.transpose_inplace()

Any preference between 1), 2), 3) or a suggestion for something else is welcome!

I think 1 would be most natural.

--
Jori Mäntysalo

Reply via email to