#18926: Auto-generated index of functions
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  ncohen                 |       Status:  needs_review
           Type:         |    Milestone:  sage-6.8
  enhancement            |   Resolution:
       Priority:  major  |    Merged in:
      Component:         |    Reviewers:
  documentation          |  Work issues:
       Keywords:         |       Commit:
        Authors:         |  3a9e3b5cbb00db4ffa5f97542ce3d027b19ee006
  Nathann Cohen          |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  public/18926           |
   Dependencies:         |
-------------------------+-------------------------------------------------

Comment (by jsrn):

 > Oh okay, it is true that somehow this function has a 'special
 behaviour'. Would you see anything wrong in filtering it out with {{{f is
 not gen_rest_table_index}}} instead of checking its module?

 No, that's probably simpler. I did the other check in case more functions
 are added to the module, but now I realise that even if that happens,
 multiple such functions are probably not imported into the same module.

 > > The function only extracts methods of a class which have been
 decorated with
 > > `@staticmethod`.
 >
 > Whaaaaat? `O_o`
 >
 > None of the methods of `IncidenceStructure` are decorated like that. And
 they appear in the list `O_o`

 Ok... It seems I've been on drugs (benadryl). I know that it was some
 class in combinat.posets that made me think this, and I though I tested
 it. I must have messed it up with the inherited-methods-not-included
 thing...

 > > This line was just to clarify the semantics of the function wrt.
 classes, since
 > > this surprised me when I first reviewed the ticket. I never intended
 > > `only_local` to have any effect when the function is called on
 classes.
 >
 > Oh. What about renaming it to 'only_local_functions` to make it
 plain(er) that it does not apply to methods?

 Yeah, that's a good idea.

--
Ticket URL: <http://trac.sagemath.org/ticket/18926#comment:20>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to