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.

Sign up for GitHub

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

Jump to bottom

modular/overconvergent/weightspace.py uses Maxima because of symbolic variables #9843

Closed
aghitza opened this issue Aug 31, 2010 · 7 comments
Closed

modular/overconvergent/weightspace.py uses Maxima because of symbolic variables #9843

aghitza opened this issue Aug 31, 2010 · 7 comments
Assignees
@craigcitro
Labels
c: modular forms p: minor / 4 t: bug
Milestone
sage-4.6

Comments

@aghitza
Copy link

aghitza commented Aug 31, 2010

This is in the top-level docstring of the file:

sage: L = Qp(17).extension(x^2 - 17, names='a'); L.rename('L')
sage: 
Exiting Sage (CPU time 0m0.94s, Wall time 0m25.72s).
Exiting spawned Maxima process.

Patch on its way.

CC: @loefflerd

Component: modular forms

Author: Alex Ghitza

Reviewer: David Loeffler

Merged: sage-4.6.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/9843
@aghitza aghitza added this to the sage-4.6 milestone Aug 31, 2010
@aghitza
Copy link
Author

aghitza commented Aug 31, 2010

comment:1

OK, I have a patch but I'm getting a trac error when trying to attach it (no space left on device).

@mwhansen
Copy link
Contributor

comment:2

Test post.

@aghitza
Copy link
Author

aghitza commented Sep 1, 2010

comment:3

Attachment: trac_9843.patch.gz

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Sep 22, 2010

Reviewer: David Loeffler

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Sep 22, 2010

Author: Alex Ghitza

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Sep 22, 2010

comment:5

Looks fine to me.

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 28, 2010

Merged: sage-4.6.alpha2

@qed777 qed777 mannequin removed the s: positive review label Sep 28, 2010
@qed777 qed777 mannequin closed this as completed Sep 28, 2010
@sagetrac-drkirkby sagetrac-drkirkby mannequin mentioned this issue Aug 31, 2010
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@craigcitro craigcitro

Labels
c: modular forms p: minor / 4 t: bug
Projects
None yet
Milestone
sage-4.6
Development

No branches or pull requests
3 participants
@mwhansen @craigcitro @aghitza