#20774: Basic singularity analysis for algebraic curves
-------------------------------------+-------------------------------------
Reporter: gjorgenson | Owner:
Type: enhancement | Status: needs_work
Priority: minor | Milestone: sage-7.3
Component: algebraic | Resolution:
geometry | Merged in:
Keywords: gsoc2016 | Reviewers: Ben Hutz
Authors: Grayson Jorgenson | Work issues:
Report Upstream: N/A | Commit:
Branch: | e6f1a9364395b0507ba872b4addf5b11d3e17ef8
u/gjorgenson/ticket/20774 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by bhutz):
At a quick glance the code changes look good, but due to traveling I won't
get to really test them for a couple days.
I forgot we need to check dimension in the curve initialization. Yes, you
could probably pass to the fractionfield in most cases and go from there
(certainly for projective curves), but with that much complication,
creating a field/ring class should be a different ticket. I consider it
low priority and can be done possibly at a later date if you decide you
have enough differentiated functionality to warrant it.
It does look to me like it is time for a point class for curves. Some
things can be done in the corresponding point class for subschemes, but I
think you have enough curve.point specific functionality to warrant its
own class. This can be a different ticket where you create the class and
add the corresponding singular analysis point functions. Most of the basic
functionality for points can just inherit directly from appropriate
subscheme point class.
--
Ticket URL: <http://trac.sagemath.org/ticket/20774#comment:8>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.