Skip to content

Conversation

@darth-cy
Copy link

Create a new specialized chip extension to handle multiple observation into the transcript in one instruction MULTI_OBSERVE.

@kunxian-xia kunxian-xia self-requested a review July 21, 2025 02:28
@darth-cy darth-cy force-pushed the feat/native_multi_observe branch from 1890560 to 3230ce8 Compare July 22, 2025 22:35
@darth-cy darth-cy force-pushed the feat/native_multi_observe branch from 3230ce8 to 831470c Compare July 22, 2025 22:42
@kunxian-xia
Copy link

I think it's more reasonable to reuse NativePoseidon2Chip for the new added opcode. Here are the reasons:

  1. we can reuse the Poseidon2SubCols columns (it has 299 columns which account for 75% of columns in NativePoseidon2Chip);

@darth-cy darth-cy changed the title Standalone MULTI_OBSERVE Chip Extend Poseidon2 Chip for MULTI_OBSERVE Aug 11, 2025
Copy link

@kunxian-xia kunxian-xia left a comment

Choose a reason for hiding this comment

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

LGTM!

@kunxian-xia kunxian-xia merged commit e4e261b into main Aug 13, 2025
0 of 27 checks passed
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.

4 participants