> Disclaimer --  I did not wrote that code and did not know it was there!

Same here :-P

> I agree that having this hidden away in graphs.py is not sensible, and
> having it in src/sage/schemes/elliptic_curves/ is more sensible.
>
> How shall we do this?  Would it work to make that a separate ticket
> from anything else you are doing, Nathann?

Yes of course. The refactoring ticket I opened will already be
sufficiently hard to review.

How do you want to see it moved there? Should we keep the syntax
Graph(....,format="elliptic_curve_congruence") or would it be better
as some_object.graph()?

Or will it become an individual function that turns some input into a graph?

It's mostly up to you, given that I do not even understand what this
code is about ^^;

Nathann

-- 
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 http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to