William Stein <wst...@gmail.com> writes: > On Mon, Jul 9, 2012 at 9:08 PM, Keshav Kini <keshav.k...@gmail.com> wrote: >> Robert Bradshaw <rober...@math.washington.edu> writes: >>> I stand by my position that it's a hack to have to break things up due >>> to technical limitations of compilation/testing. Maybe a necessary >>> hack, but we've contorted the code due to tool limitations, not >>> because it's a good way to organize things. (Perhaps things could be >>> laid out more logically, but having authored much of the code in >>> question, I still couldn't tell you without looking it up if >>> Matrix.augment() was in matrix0, matrix1, or matrix2.) >> >> +1 to all this, for what it's worth. >> >> -Keshav > > Seriously? You guys actually *want* a single 20,000 line Cython file? > Why? The only argument you are giving is that you can't remember > which file something is in, so let's just put everything in one > massive file. I'm clearly missing something.
Well, in Python, the structure of the files you have is supposed to more or less directly reflect the hierarchical structure of your package in terms of its modules and submodules, right? Splitting up a large file into multiple smaller files simply because it's large, and doing the splitting up more or less arbitrarily while trying to merge the contents of the new files into the same namespace with imports, screams "hack" to me. Of course, practical considerations might overrule this screaming voice, but it's still there :) -Keshav ---- Join us in #sagemath on irc.freenode.net ! -- -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
