Skip to content

Commit c573282

Browse files
authoredAug 28, 2023
chore: use 2^19 as MAX_CIRCUIT_SIZE for NodeJS CLI (#1834)
resolves #1829 # Checklist: Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge. - [ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag. - [ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code. - [ ] Every change is related to the PR description. - [ ] I have [linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) this pull request to relevant issues (if any exist).
1 parent 58b3c3a commit c573282

File tree

1 file changed

+7
-2
lines changed
  • circuits/cpp/barretenberg/ts/src

1 file changed

+7
-2
lines changed
 

‎circuits/cpp/barretenberg/ts/src/main.ts

+7-2
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,13 @@ import { Command } from 'commander';
88
createDebug.log = console.error.bind(console);
99
const debug = createDebug('bb.js');
1010

11-
// Maximum we support natively. It is 2^19 for browser.
12-
const MAX_CIRCUIT_SIZE = 2 ** 23;
11+
// Maximum we support in node and the browser is 2^19.
12+
// This is because both node and browser use barretenberg.wasm.
13+
//
14+
// This is not a restriction in the bb binary and one should be
15+
// aware of this discrepancy, when creating proofs in bb versus
16+
// creating the same proofs in the node CLI.
17+
const MAX_CIRCUIT_SIZE = 2 ** 19;
1318

1419
function getBytecode(bytecodePath: string) {
1520
const encodedCircuit = readFileSync(bytecodePath, 'utf-8');

0 commit comments

Comments
 (0)
Please sign in to comment.