Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implement the projective dual of a plane curve #8801

Closed
aghitza opened this issue Apr 28, 2010 · 20 comments
Closed

implement the projective dual of a plane curve #8801

aghitza opened this issue Apr 28, 2010 · 20 comments

Comments

@aghitza
Copy link

aghitza commented Apr 28, 2010

This was requested by Ursula Whitcher on sage-support. She adds:

  I'm playing with a family of plane curves with rational coefficients in 
  the complex projective plane.  So rational or complex numbers would be 
  enough for me to test examples.  In a perfect world I'd be able to 
  specify a family using rational functions of arbitrary constants 
  (something like a x^2 + b/(a-1) y^2), and compute the projective dual in 
  terms of those constants.

A good place to start in implementing this could be http://www.emilvolcheck.com/dual.ps

Component: algebraic geometry

Keywords: projective duality

Author: David Eklund

Branch/Commit: 5a4b960

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/8801

@sagetrac-davideklund
Copy link
Mannequin

sagetrac-davideklund mannequin commented May 26, 2012

comment:1

The attached patch implements this for (reduced and irreducible) hypersurfaces over the rationals. I intend to generalize this.

I use Grobner bases and elimination. Resultants might be faster so I might switch to that approach.

If you plug in a variety I think the dual should be reduced. I'm not sure exactly what the scheme structure of the output is at the moment.

Something related to think about for the general case: given a subscheme X of projective space, what should the dual of X be?

I will look at the approach described in the attached paper. When the dual is a hypersurface and has smaller degree than "expected", that approach seems to be better than the one used at the moment.

@sagetrac-davideklund
Copy link
Mannequin

sagetrac-davideklund mannequin commented May 26, 2012

Author: David Eklund

@sagetrac-davideklund
Copy link
Mannequin

sagetrac-davideklund mannequin commented Aug 24, 2012

Attachment: trac_8801_projective_duals.patch.gz

The patch implements duals of projective hypersurfaces. Patch rebased on top of Sage 5.2.

@sagetrac-davideklund
Copy link
Mannequin

sagetrac-davideklund mannequin commented Aug 24, 2012

comment:3

Patch rebased on top of Sage 5.2.

@fchapoton
Copy link
Contributor

Commit: 04c7bd0

@fchapoton
Copy link
Contributor

Branch: u/chapoton/8801

@fchapoton
Copy link
Contributor

comment:4

Here is a git branch. Maybe this could be considered as need review ?


New commits:

50eb935Trac 8801: Add duals of projective hypersurfaces.
04c7bd0trac #8801 make sure that tests pass

@sagetrac-davideklund
Copy link
Mannequin

sagetrac-davideklund mannequin commented Mar 6, 2014

comment:5

I guess it could be reviewed. It could be useful as it is. I had some plans to do something better but someone stole all my time... There is no particular reason to stick to hypersurfaces, it was just meant as an initial simplification.

@sagetrac-davideklund
Copy link
Mannequin

sagetrac-davideklund mannequin commented Mar 6, 2014

comment:6

Nor is there any particular reason to stick to varieties over Q.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 27, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

d8cb633Merge branch 'u/chapoton/8801' into 6.5.b6

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 27, 2015

Changed commit from 04c7bd0 to d8cb633

@fchapoton
Copy link
Contributor

comment:8

turning that into needs review to require feedback:

what is the class of rings over which this can currently work ?

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 27, 2015

Changed commit from d8cb633 to ba23a27

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 27, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

4aa3fb9Merge branch 'u/chapoton/8801' into 6.6.b1
ba23a27trac #8801 correct use of elim lib of singular

@fchapoton fchapoton added this to the sage-6.7 milestone Apr 29, 2015
@fchapoton
Copy link
Contributor

Changed keywords from none to projective duality

@fchapoton fchapoton modified the milestones: sage-6.7, sage-6.8 May 28, 2015
@fchapoton fchapoton modified the milestones: sage-6.8, sage-6.9 Aug 9, 2015
@fchapoton
Copy link
Contributor

comment:13

nobody interested by this ticket ?

@fchapoton fchapoton modified the milestones: sage-6.9, sage-6.10 Oct 18, 2015
@fchapoton fchapoton modified the milestones: sage-6.10, sage-7.0 Jan 6, 2016
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 18, 2016

Changed commit from ba23a27 to 5a4b960

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 18, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

5a4b960Merge branch 'u/chapoton/8801' into 7.0.rc1

@fchapoton fchapoton modified the milestones: sage-7.0, sage-7.1 Jan 21, 2016
@vbraun
Copy link
Member

vbraun commented Feb 23, 2016

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Feb 24, 2016

Changed branch from u/chapoton/8801 to 5a4b960

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants