Skip to content

Conversation

bleepbloopsify
Copy link
Contributor

poc pull request for CI

@bleepbloopsify bleepbloopsify self-assigned this Oct 3, 2025
@bleepbloopsify bleepbloopsify force-pushed the leon/zkdriver branch 2 times, most recently from 238d222 to 158a5b9 Compare October 3, 2025 20:09
@bleepbloopsify bleepbloopsify changed the base branch from main to leon/formatting-upgrades October 3, 2025 20:09
Copy link
Contributor Author

@bleepbloopsify bleepbloopsify left a comment

Choose a reason for hiding this comment

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

hmm, I can see that numChunks is only respected from the ZkProgram constructor, not the actual compile call. I omitted it here, but that might be incorrect

forceRecompile?: boolean;
proofsEnabled?: boolean;
withRuntimeTables?: boolean;
numChunks?: number;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't use this config option?

@bleepbloopsify bleepbloopsify force-pushed the leon/formatting-upgrades branch 7 times, most recently from 1fd7766 to 676a195 Compare October 7, 2025 18:03
Base automatically changed from leon/formatting-upgrades to main October 7, 2025 18:44
@bleepbloopsify
Copy link
Contributor Author

proposal has been run by @Trivo25 , will graduate this to a full-fledged design document

the bug will be addressed in a separate PR

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.

1 participant