On Mon, May 22, 2017 at 4:01 PM, Jori Mäntysalo <jori.mantys...@uta.fi> wrote: > 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.
Though I'd also add that something like 2) makes perfect sense as an *implementation* detail. -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at https://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.