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

fix "from ... import *" in class QuadraticForm (cf #4470) #5403

Closed
tornaria opened this issue Feb 28, 2009 · 9 comments
Closed

fix "from ... import *" in class QuadraticForm (cf #4470) #5403

tornaria opened this issue Feb 28, 2009 · 9 comments

Comments

@tornaria
Copy link
Contributor

There are a number of "from ... import *" in the new quadratic forms code. This messes up tab completion (among other issues).

The attached patch fixes the issue by explicitly listing every function defined in the quadratic forms files.

Component: quadratic forms

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

@tornaria
Copy link
Contributor Author

Attachment: patch.QF.fix_import_star.gz

patch to fix import * issues in quadratic forms

@tornaria
Copy link
Contributor Author

comment:1

There are still some issues in the attached patch. I'm working on that now.

@tornaria
Copy link
Contributor Author

Attachment: patch.QF.fix_import_star.2nd.gz

2nd version of the patch, stilll some issue with deepcopy

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 1, 2009

comment:2

This would be nice to fix since on every first startup after moving Sage or cloning python will complain about those imports.

I am also marking the patch as needs work until it is ready for review. That way the patch will not be lost :)

Cheers,

Michael

@tornaria
Copy link
Contributor Author

tornaria commented Mar 1, 2009

Attachment: patch.QF.fix_import_star.3rd.gz

3rd version of the patch, tests pass now

@tornaria
Copy link
Contributor Author

tornaria commented Mar 1, 2009

comment:3

In the 3rd patch I added a fix for issue with copying and caching which for some weird reason didn't show up before.

All quadratic_form tests pass with this patch applied (NOT incremental, just get the 3rd one).

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 2, 2009

comment:5

Positive review.

Cheers,

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 2, 2009

comment:6

Merged in Sage 3.4.rc0.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Mar 2, 2009
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Mar 2, 2009

Attachment: trac_5403_patch.QF.fix_import_star.3rd.patch.gz

This is a properly named version of Gonzalo's third patch

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

1 participant