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.

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

feat: Add support for calculating square root of a fixed point float #8

Merged
merged 4 commits into from
Dec 10, 2023

Conversation

alan890104
Copy link
Contributor

This pull request adds a new feature to the math library that allows for calculating the square root of a fixed point float. It includes the implementation of the sqrtFloat function, along with tests and a CI workflow for automated testing.

* feat(math_example.tact): add support for calculating square root of a fixed point float
* feat(float.fc): implement sqrtFloat function to calculate the square root of a fixed point float
* test(float.spec.ts): add tests for calculating square root of a fixed point float
Copy link
Contributor

@ipromise2324 ipromise2324 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The way you solve sqrtFloat() is cooooooool !!!

Copy link
Contributor

@WaterSo0910 WaterSo0910 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! I'm really impressed with the progress and the quality of the code. I've added some comments within the code for further clarification and suggestions. Could you please take a look and address them when you have a moment? Your expertise and insights on these matters would be highly appreciated. Thanks for your excellent contribution!

contracts/packages/math/float.fc Show resolved Hide resolved
contracts/packages/math/float.fc Show resolved Hide resolved
@alan890104 alan890104 merged commit 45757ac into main Dec 10, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants