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

make _multiple_x_*() methods work for all n≠0 #35035

Merged
merged 1 commit into from
Mar 3, 2023
Merged

make _multiple_x_*() methods work for all n≠0 #35035

merged 1 commit into from
Mar 3, 2023

Conversation

yyyyx4
Copy link
Member

@yyyyx4 yyyyx4 commented Feb 9, 2023

Currently, these methods expect $n\geq2$ for no real reason. This patch makes them work for all $n\neq0$.

@yyyyx4 yyyyx4 changed the title make _multiple_x_*() methods work for all n make _multiple_x_*() methods work for all n≠0 Feb 9, 2023
@JohnCremona JohnCremona self-requested a review February 10, 2023 16:52
@vbraun vbraun merged commit a822a21 into sagemath:develop Mar 3, 2023
@mkoeppe mkoeppe added this to the sage-10.0 milestone Mar 3, 2023
@yyyyx4 yyyyx4 deleted the public/make_multiple_x_methods_work_for_all_n branch March 3, 2023 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JohnCremona JohnCremona JohnCremona approved these changes

Assignees
No one assigned
Labels
c: elliptic curves p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-10.0
Development

Successfully merging this pull request may close these issues.
4 participants
@yyyyx4 @JohnCremona @vbraun @mkoeppe