-
Notifications
You must be signed in to change notification settings - Fork 45
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
Create ComplexFloat trait to model Floats as a subset of Complex<T> #95
Conversation
If you want it you could add a |
I will certainly do so. Thanks! |
…point and complex numbers. Added a couple of methods to Complex that were present in Float but missing in Complex. Added tests for many methods on the new trait and for all new methods on Complex. Resolves rust-num#2. Authored originally by Martin Reboredo <yakoyoku@gmail.com> Updated by Josiah Bills <josiah@adoniram.net>
Ping, any chance I could get a review before end of year? |
Bump |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry that I've left this neglected for so long...
…ealing ComplexFloat.
Awesome work, I'm happy this trait comes alive. |
I tweaked a few more things, but I think we're good. 🤞 Thanks! bors r+ |
Build succeeded: |
Resolves #2. Replaces #53. Nearly all the work here was already done by YakoYakoYokuYoku. I have merely updated it to master and made a few discretionary changes which I think improve the work slightly. More unit tests will be coming.