Skip to content

feat: add blake3 option for second public input - #522

Open
VanhGer wants to merge 6 commits into
ProjectZKM:v1.2.5-blake3from
VanhGer:blake3
Open

feat: add blake3 option for second public input#522
VanhGer wants to merge 6 commits into
ProjectZKM:v1.2.5-blake3from
VanhGer:blake3

Conversation

@VanhGer

@VanhGer VanhGer commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

No description provided.

/// feature is enabled.
/// By default, the variable is disabled.
pub fn zkm_imm_wrap_vk_mode() -> bool {
let value = std::env::var("ZKM_IMM_WRAP_VK").unwrap_or_else(|_| "false".to_string());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

do we have to use std in the zkm::primitives? any other way to distinguish whether ZKM_IMM_WRAP_VK is enabled?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

zkm-primitives is already a plain std crate everywhere, not no_std.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

right now it just has only way which checks the env ZKM_IMM_WRAP_VK is set to 1 or 0.

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.

2 participants