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

Directly include parallel math #5

Merged
merged 51 commits into from
Jun 10, 2024

Conversation

AndreyMlashkin
Copy link
Contributor

No description provided.

Zerg1996 and others added 30 commits May 31, 2022 21:49
…R_32

Moving changes up to PR #32 from crypto3 math library
…R_32

Some minor changes to the actor math library related to the latest updates from the crypto3 lib
* Move the pow function with its tests.

* Fixes to pow function according to review comments.
…#6)

* Merge pull request #42 from NilFoundation/41-fix-operator-=-for-polynomial_dfs

Fixed operator-= for polynomial_dfs.

* Merge pull request #45 from NilFoundation/44-add-a-function-zero-to-polynomial-classes

44 add functions zero() and one() to polynomial classes

* Merge pull request #46 from NilFoundation/small_fix_in_function_one

Fixing a small bug in polynomial_dfs function one()

* Adding a hash function.

* Hashing moved from std namespace.

* Fixing some bugs and optimziing polynomial operations.

* Adding operators.

* Fixes and cleanup.

* Merge pull request #54 from NilFoundation/53-mathpolynomial-division-on-one-polynomial-bugs

Fixing polynomial division over a poly of degree 0.

* Merge pull request #57 from NilFoundation/56-add-the-missing-reffernces

Adding missing references.

* Few final fixes.

---------

Co-authored-by: ETatuzova <e.tatuzova@nil.foundation>
Co-authored-by: root <mail.nikita.kaskov@gmail.com>
… 4 cores are available. Also divide work between cores more equally (#8)

* Changing function block_execution, don't utilize core 0 if at least 4 cores are available.
* Improve parallelization of fft.
…fb793896c162ecc34df79d779 (#17)

* Add consistent testing (#65)

* Cleaning the code of evaluation domains. [SyncWith: crypto3-zk#201] (#64)

* Cleaning the code of evaluation domains a bit.

* Small fixes, now all tests pass.

---------

Co-authored-by: x-mass <36629999+x-mass@users.noreply.github.com>
…vector

Added function block_execution_vector with desired effect #18
martun and others added 21 commits February 5, 2024 20:08
…46f7466c09dbc3731b (#23)

* Adding a performance test for FFT. (#69)

* Added FFT cache to domains [SyncWith: crypto3-zk#277] (#73)

Added caching to basic_radix2_domain_aux and it's calls.

* Fixing fft test.

---------

Co-authored-by: Iluvmagick <d.tabalin@nil.foundation>
* New actor math.
* Add benchmark test (with reusable macro)

* Introduce polynomial_sum for fast batched addition
Copy link

Linux Test Results

 50 files  ±0   50 suites  ±0   1m 39s ⏱️ -1s
175 tests ±0  170 ✅ ±0   5 💤 ±0  0 ❌ ±0 
398 runs  ±0  388 ✅ ±0  10 💤 ±0  0 ❌ ±0 

Results for commit c01f26d. ± Comparison against base commit d09f687.

Copy link

Mac Test Results

 25 files  ±0   25 suites  ±0   27s ⏱️ -1s
175 tests ±0  170 ✅ ±0  5 💤 ±0  0 ❌ ±0 
199 runs  ±0  194 ✅ ±0  5 💤 ±0  0 ❌ ±0 

Results for commit c01f26d. ± Comparison against base commit d09f687.

@AndreyMlashkin AndreyMlashkin merged commit 3227dd3 into master Jun 10, 2024
5 checks passed
@x-mass x-mass deleted the directly-include-parallel-math branch June 11, 2024 09:47
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.

7 participants