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

mod(::Complex, ::Real) and similar methods #37376 #37377

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

mod(::Complex, ::Real) and similar methods #37376 #37377

wants to merge 2 commits into from

Conversation

chakravala
Copy link
Contributor

@chakravala chakravala commented Sep 3, 2020 •
edited by stevengj

preliminary commit to fix #37376

stevengj
stevengj reviewed Sep 3, 2020
View reviewed changes
base/complex.jl Outdated Show resolved Hide resolved
@stevengj stevengj added needs news A NEWS entry is required for this change needs tests Unit tests are required for this change domain:complex Complex numbers labels Sep 5, 2020
@ViralBShah ViralBShah added the domain:maths Mathematical functions label Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stevengj stevengj stevengj left review comments

Assignees
No one assigned
Labels
domain:complex Complex numbers domain:maths Mathematical functions needs news A NEWS entry is required for this change needs tests Unit tests are required for this change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

mod(::Complex{<:Integer}, ::Integer) and div etc.
3 participants
@chakravala @stevengj @ViralBShah