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
count_points() error for generic schemes. (includes patch) #11526
Comments
Attachment: trac_11526_solves_error.patch.gz |
This comment has been minimized.
This comment has been minimized.
comment:3
The patch makes sense, it fixes a bug in the method points() of the class SchemeHomset_affine_coordinates. Note: the same typo (Z instead of ZZ) seems to occur in the method points() of the class SchemeHomset_projective_coordinates_ring, which is not implemented yet. |
In addition to fixing the bug, the patch includes a doctest illustrating this. |
comment:4
Attachment: trac_11526_bugfix_with_doctest.patch.gz The patch trac_11526_bugfix_with_doctest.patch was made using Sage 4.7.1. |
This comment has been minimized.
This comment has been minimized.
comment:5
Now that you're working on the docstring anyway, it would be good to add INPUT and OUTPUT sections to it, as is the standard convention. |
Attachment: trac_11526_bugfix_with_docstring.patch.gz In addition to earlier changes, the patch includes INPUT and OUTPUT info. |
comment:6
The patch trac_11526_bugfix_with_docstring.patch was made using Sage 4.7.1. |
This comment has been minimized.
This comment has been minimized.
The same patch as above together with the fix in the projective case. |
Attachment: trac_11526_bugfix_with_docstring.2.patch.gz fixes the bug (also for the projective case), docstring with Input and Output. |
Changed author from Miguel Marco to Miguel Marco, davideklund, Frithjof Schulze |
This comment has been minimized.
This comment has been minimized.
Reviewer: Johan Bosman, Julian Rueth |
comment:7
Attachment: trac_11526_final.patch.gz Apply trac_11526_final.patch Tested against sage-4.8.alpha3 and doctests pass. attachment: trac_11526_final.patch is identical to [trac_11526_bugfix_with_docstring.2.patch] except for the commit message which I replaced with something that makes hopefully more sense to somebody who does not know this ticket. We still don't have a docstring for SchemeHomset?_projective_coordinates_ring.points() but I guess that is acceptable since there wasn't any before and it's not implemented anyway. I'm unsure if "Reviewers" should only mention the person who clicked "positive review" or everybody who was part of the review process. Anyway, I added some authors and reviewers, I hope nobody minds. Btw. davideklund, your name is not in the AccountNamesMappedtoRealNames on the trac wiki start page. |
Merged: sage-4.8.alpha4 |
Changed author from Miguel Marco, davideklund, Frithjof Schulze to Miguel Marco, David Eklund, Frithjof Schulze |
sage raises an error when trying to count the number of points of a subscheme of an affine scheme.
See, for instance:
Apply:
attachment: trac_11526_final.patch
Component: algebraic geometry
Author: Miguel Marco, David Eklund, Frithjof Schulze
Reviewer: Johan Bosman, Julian Rueth
Merged: sage-4.8.alpha4
Issue created by migration from https://trac.sagemath.org/ticket/11526
The text was updated successfully, but these errors were encountered: