On Mon, May 30, 2016 at 9:47 AM, Jori Mäntysalo <jori.mantys...@uta.fi> wrote: > On Mon, 30 May 2016, Erik Bray wrote: > >>> Imho many sage modules have grown too long, including but not limited to >>> some 20kloc monsters. +1 to splitting things up! >> >> >> Yikes! I'm not sure I've even run into any of those yet. 20kloc is >> definitely worth splitting up. ~4k is a lot more arguable I think, >> but I personally don't like files to get much longer than 1kloc >> depending of course on how practical it is to further subdivide, which >> it isn't always. > > > What you mean by kloc? Lines of *code*, or lines of whole file? > > There is 21312 lines at src/sage/graphs/generic_graph.py. But there just is > so much functions defined for (di)graphs, with different options and > algorithms and so on.
Definitely the intention was just file length. For example, https://github.com/sagemath/sage/blob/master/src/sage_setup/autogen/interpreters.py is *mostl* (maybe 80%) comments/docstrings! It's some of the best-written code I've ever seen. -- William (http://wstein.org) -- 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.
