-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Pure fixed-point math functions #1543
Comments
@mingjuu You can :) I'll assign you right now and if you have any questions, feel free to ask here or in chat. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
@heinezen Thank! I will touch you if I have any questions. |
Hello, Is there anything left to do here that I can contribute to? |
@adamresemius Yes, the unticked boxes in the issue description is what's not implemented at the moment. |
I think I know how to implement the other functions in fpm. I have pow already on mine. |
Hey, I'm a beginner and could try my hand at implementing sqrt (and possibly the others after) if it isn't being worked on right now |
@firednerve You are welcome to go for it :) |
Thanks! I’ll ask if I have any questions. |
Hi! Is this issue still open to complete the "nice to have" operations? I do not see the implementations added and would like to try it out as someone who is new to open source! |
Hi there, sin, cos and tan is already implemented in this PR, same for sqrt and atan2, already implemented |
already created the missing functions and added missing test cases. |
Required Skills: C++
Difficulty: Easy
We should add additional math operations to our fixed-point implementation, so that all calculations can be done purely deterministic. The
fpm
library already has these operations and could be used as an inspiration.Important here: These operations must not use any floating point arithmetic for the calculation. They must receive fixed-point values as input and must also return a fixed-point value. This means
std
library calculations should not be used for solving this issue.Interesting for us:
Nice to have:
Further Reading
The text was updated successfully, but these errors were encountered: