Changes between Initial Version and Version 1 of Ticket #14119


Ignore:
Timestamp:
02/15/13 04:08:24 (10 years ago)
Author:
kcrisman
Comment:

"Authors" is for author of a patch; you're already the reporter :-)

But

sage: import_statements(DiGraph)
from sage.graphs.digraph import DiGraph

so maybe that's okay?

A few things also give "class" instead of this, a few (colors) give dictionaries, and don't even try copying :-) (since sage: print "%s"%copying ends up printing the entire license statement).


Basically, we probably need a little more uniformity in that some things do not have a __name__. That's what happens here.

sage: qepcad_formula.__module__
'sage.interfaces.qepcad'
sage: qepcad_formula
<sage.interfaces.qepcad.qepcad_formula_factory instance at 0x103b37290>

So this might need a bit more work than just fixing one thing.


I also note that this doesn't actually return a string, but simply prints the imports and then returns None. I don't know if I like that.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14119

    • Property Priority changed from major to minor
    • Property Component changed from PLEASE CHANGE to misc
    • Property Authors changed from chrisjamesberg to
  • Ticket #14119 – Description

    initial v1  
    11Eric Rowland pointed out the following weird behavior with respect to import_statements.
    22
    3 sage: import_statements(digraph)
     3{{{
     4sage: import_statements(digraphs)
    45
    56from sage.graphs.digraph_generators import <sage.graphs.digraph_generators.DiGraphGenerators instance at 0x115bf4b00>
     7}}}