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

Updated placeholder and hashes. #321

Merged
merged 2 commits into from
May 7, 2024
Merged

Updated placeholder and hashes. #321

merged 2 commits into from
May 7, 2024

Conversation

ETatuzova
Copy link
Contributor

… bug in evaluation points#320

@ETatuzova ETatuzova linked an issue Apr 23, 2024 that may be closed by this pull request
Changes related to external placeholder verifiers #320
Closed
@ETatuzova ETatuzova changed the title Moved large function from lookup_argument to constraint_system, fixed… Placeholder profiling updated types Apr 23, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 23, 2024 •
edited
Loading

Test Results

  30 files    30 suites   1m 10s ⏱️
  79 tests   79 ✔️ 0 💤 0 ❌
168 runs  168 ✔️ 0 💤 0 ❌

Results for commit 1386e08.

♻️ This comment has been updated with latest results.

@ETatuzova ETatuzova changed the title Placeholder profiling updated types Updated placeholder and hashes [SyncWith:crypto3-zk#321] Apr 26, 2024
@ETatuzova ETatuzova force-pushed the 320-placeholder-profiling branch from 05312ce to 46fee8d Compare April 26, 2024 22:21
@nil-foundation-cicd-bot
Copy link

nil-foundation-cicd-bot bot commented Apr 26, 2024 •
edited by martun
Loading

PRs, synced with the current issue:

Note: Please do not edit this comment; it's generated automatically.

@nil-foundation-cicd-bot
Copy link

Tests will be run in sync with other PRs containing [SyncWith NilFoundation/crypto3-zk#321] in title. You can find related PRs as linked with mentioned issue.

This was referenced Apr 26, 2024
Merged
Closed
Merged
Merged
Merged
@ETatuzova ETatuzova requested a review from martun April 26, 2024 22:59
@x-mass x-mass force-pushed the 320-placeholder-profiling branch from 46fee8d to dd8b7d9 Compare April 29, 2024 07:48
@x-mass x-mass force-pushed the 320-placeholder-profiling branch from a6f0885 to c333f0f Compare April 30, 2024 00:14
@x-mass x-mass force-pushed the 320-placeholder-profiling branch from c333f0f to a96e603 Compare May 1, 2024 08:18
@martun martun changed the title Updated placeholder and hashes [SyncWith:crypto3-zk#321] Updated placeholder and hashes [Sync Wit h:crypto3-zk#321] May 2, 2024
@martun martun force-pushed the 320-placeholder-profiling branch from 8e7b6d3 to 1386e08 Compare May 7, 2024 06:56
@martun martun changed the title Updated placeholder and hashes [Sync Wit h:crypto3-zk#321] Updated placeholder and hashes. May 7, 2024
@martun martun force-pushed the 320-placeholder-profiling branch from 1386e08 to dad75fa Compare May 7, 2024 12:33
@martun martun force-pushed the 320-placeholder-profiling branch from dad75fa to f254a0a Compare May 7, 2024 12:34
@martun martun merged commit f8f66ae into master May 7, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martun martun martun approved these changes

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

Successfully merging this pull request may close these issues.

Changes related to external placeholder verifiers
3 participants
@ETatuzova @martun @x-mass