Skip to content

feat!: Configure full RUS state factory in Iceberg builder - #520

Merged
maximilianruesch merged 5 commits into
mainfrom
mr/feat/state-factory-iceberg
Nov 4, 2026
Merged

maximilianruesch merged 5 commits into
mainfrom
mr/feat/state-factory-iceberg

Conversation

@maximilianruesch

@maximilianruesch maximilianruesch commented Nov 4, 2026 •

Copy link
Copy Markdown
Collaborator

BREAKING CHANGE: The Iceberg builder now accepts a full RUS state factory configuration instead of just a max_attempts integer.

@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

This PR contains breaking changes to the public Python API.

Breaking changes summary
src/guppyft/code/iceberg/encode.py:0: IcebergBuilder.with_state_prep_rus_attempts:
Public object was removed

src/guppyft/code/iceberg/encode.py:0: IcebergBuilder.__init__(_options):
Parameter default was changed:
Old: _IcebergOptions(qed_policy=(IcebergQEDPolicy()), syndrome_during_measurement=False, state_prep_rus_attempts=3, dynamic_qubit_reuse=(DynamicQubitReuse.enabled))
New: _IcebergOptions(qed_policy=(IcebergQEDPolicy()), syndrome_during_measurement=False, state_factory_conf=(RUSStateFactoryConf(size=1, max_attempts=3)), dynamic_qubit_reuse=(DynamicQubitReuse.enabled))


@maximilianruesch
maximilianruesch marked this pull request as ready for review November 4, 2026 14:51
@maximilianruesch
maximilianruesch requested a review from a team as a code owner November 4, 2026 14:51
@maximilianruesch
maximilianruesch requested review from cqc-alec and removed request for hsemenenko November 4, 2026 15:04
Comment thread src/guppyft/code/iceberg/encode.py Resolved
@maximilianruesch
maximilianruesch merged commit ea6b73a into main Nov 4, 2026
21 checks passed
@maximilianruesch
maximilianruesch deleted the mr/feat/state-factory-iceberg branch November 4, 2026 15:31
@hugrbot hugrbot mentioned this pull request Nov 4, 2026
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