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

Directly include parallel crypto3 #5

Merged
merged 424 commits into from
Sep 9, 2024
Merged

Directly include parallel crypto3 #5

merged 424 commits into from
Sep 9, 2024

Conversation

AndreyMlashkin
Copy link
Collaborator

No description provided.

martun and others added 30 commits March 20, 2023 06:15
@martun
Moving 23 commits from crypto3-zk
aaba24a
@martun
Few fixes
4f2300b
@martun
Few fixes, now everything works
0b4b597
@martun
Fixing a few tests.
1b037c7
@martun
Fixing a test
010aa44
@martun
Changes related to review comments.
69f0855
@martun
Merge pull request #1 from NilFoundation/crypto3_math_changes_up_to_P…
c10b3a3 
…R_32

Moving changes up to PR #32 from crypto3 math library
@martun
Some actor_math functions now return futures, so need to add .get()
d16bd6c
@martun
Few small fixes, trying to make the tests pass.
ecaa39b
@martun
Fixing the tests. They still crash due to multithreading.
15bfee1
@martun
Fixes to the test.
fe535f3
@martun
Some minor changes to the actor math library.
f60b14c
@Zerg1996
Merge pull request #2 from NilFoundation/crypto3_math_changes_up_to_P…
35408f4 
…R_32

Some minor changes to the actor math library related to the latest updates from the crypto3 lib
@martun
Some more fixes.
7b98189
@martun
Another small fix
33a4029
@Zerg1996
Merge pull request #1 from NilFoundation/48-crypto3-zk-changes-23-com…
ad29930 
…mits

Moving commits from crypto3-zk, up to PR #120, commit c22778b (23 commits total)
@martun
I forgot .get() in a few places during the last change. (#2)
e2a64e1
@martun
Adding ACTOR_FIXTURE_TEST_CASE, and fixing an unused variable issue. (#8
3a91181 
)

* Adding ACTOR_FIXTURE_TEST_CASE, and fixing an unused variable issue.

* Changes according to review comments, removing duplicate code.

* Adding the forgotten define.
@martun
Draft: 3 Moving changes from crypto3-zk up to commit 14d85b9. (#4)
0bf64ac 
* Moving changes from crypto3-zk.

* Removing unnneded changes.

* Few last fixes

* This class was commented out in the crypto3 version.
@nemothenoone
Dependencies lookup changes.
702789d
@NickVolynkin
Use http paths for submodules
efa66dc 
Required for building proof-market-toolchain in CI.
@martun
Move the pow function from crypto3-math. (#4)
78fe0e5 
* Move the pow function with its tests.

* Fixes to pow function according to review comments.
@martun
Move changes from crypto3-zk up to commit a9b351b (#7)
0172596 
* Move changes from crypto3-zk up to commit a9b351b.

* Commenting disabled tests.

* Small changes related to review comments.
@martun
Trying to fix the pipelines. (#138)
95f056a
@martun
139 refactor placeholder proof system (#140)
8667d29 
* Refactored placeholder code a bit, more changes will come.

* Refactoring Placeholder to reduce memory usage, and clear the code.
@martun
Merge similar terms in non-linear-combination, and make everything ha…
f9534fb 
…shable. (#147)

* Making everything hashable, we need it for later changes. Also merging similar terms when creating a non-linear combination.
@martun
Fixing compilation on some compilators due to std namespace. (#152)
6472c5f
@martun
removing namespace std from expression.hpp (#153)
f56bd0a
@martun
Changes after poseidon changes are merged. (#155)
9becaa2
@martun
145 make gate argument one large expression (#151)
17cf0fc 
* Some initial changes.

* Saving changes to rebase.

* Saving changes to rebase.

* Now everything works!

* Some minor change.

* Fixes, now everything works.

* Small changes.
AndreyMlashkin and others added 22 commits July 18, 2024 11:45
@AndreyMlashkin @vo-nil
cleanup cmake polices
3d36934
@x-mass @vo-nil
Fix proof producer bug for Boost 1.83
f1d4414
@martun @vo-nil
Remove failing tests from cmake files.
73d259b
@martun @vo-nil
Disable slow and failing tests.
409a5ae
@martun @vo-nil
Fix multiprecision for mac, there's a difference related to std::vect…
61cb3a8 
…or<bool>
@vo-nil
Reworked test for kzg commitment #281
f34d4fa
@martun @vo-nil
Fix tests on ci and move slow tests to bench_tests folder.
52131eb
@vo-nil
Removed bogus test #285
8bb48d3
@martun @vo-nil
Fixed comparison of numbers of different bit length.
9b682df
@vo-nil
Synchronize changes from crypto3, fix broken tests
c0e5955 
Corrected file permissions #15

Removed missing target from flake.nix #15
@vo-nil
Changed interface for proof_of_work #11
61c1cf4 
Added parallel implementation for proof_of_work for non-poseidon trascript, #11

Update for grinding, synchronizing with crypto3 #11

Renamed int_be to to_byte_array #11
@x-mass
chore: nix flake dependencies updated
404077b
@martun
Parallelize gate argument outside the expression tree. (#19)
bcaebf3 
* Parallelize gate argument outside the expression tree.

* Optimize memory consumption a bit.

---------

Co-authored-by: Martun Karapetyan <[email protected]>
@martun
Move nix related changes from crypto3 to parallel-crypto3. (#21)
9db0308
@AndreyMlashkin
update flake
fa2f404
@AndreyMlashkin
Move all files into ./parallel-crypto3/ directory
732a2d0
@AndreyMlashkin
delete unused files
6e289a5
@AndreyMlashkin
directly include parallel-crypto3
a6be440
@AndreyMlashkin
correct adding parallel-crypto3
fe84fbf
@AndreyMlashkin
implement parallel-crypto3 flake
b9c7e6c
@AndreyMlashkin
fix merkle test
9c00a6d
@AndreyMlashkin
optimize flake
2a353cc
akokoshn
akokoshn reviewed Sep 6, 2024
View reviewed changes
Copy link
Collaborator

@akokoshn akokoshn left a comment

Choose a reason for hiding this comment

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

let's make single one .clang-format for all repository

parallel-crypto3/parallel-crypto3.nix Outdated Show resolved Hide resolved
parallel-crypto3/parallel-crypto3.nix Show resolved Hide resolved
akokoshn
akokoshn approved these changes Sep 7, 2024
View reviewed changes
Copy link
Collaborator

@akokoshn akokoshn left a comment

Choose a reason for hiding this comment

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

could you please remove empty parallel-crypto3/.gitmodules

@AndreyMlashkin AndreyMlashkin merged commit 00e0a64 into master Sep 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akokoshn akokoshn akokoshn approved these changes

@knazarov knazarov Awaiting requested review from knazarov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.