Skip to content

Review fixissue755#768

Open
PabloCarmona wants to merge 4 commits intomasterfrom
review-fixissue755
Open

Review fixissue755#768
PabloCarmona wants to merge 4 commits intomasterfrom
review-fixissue755

Conversation

@PabloCarmona
Copy link
Copy Markdown
Collaborator

Related issues

Closes #755
PR #761

Description

Review for PR #761 and add a new regression test in tests/test_specific_tiles.py (line 154) that exercises a negative reset bias through the existing TransferCompound decay path.

@PabloCarmona PabloCarmona force-pushed the review-fixissue755 branch 2 times, most recently from 793458c to 36458c7 Compare April 10, 2026 10:51
@PabloCarmona PabloCarmona requested a review from bayo-ibm April 10, 2026 10:52
cyansky99 and others added 4 commits April 10, 2026 13:13
Signed-off-by: cyansky99 <1999.cyan@gmail.com>
Signed-off-by: Pablo Carmona Gonzalez <pablocarmonagonzalez@gmail.com>
Signed-off-by: Pablo Carmona Gonzalez <pablocarmonagonzalez@gmail.com>
…phery

Signed-off-by: Pablo Carmona Gonzalez <pablocarmonagonzalez@gmail.com>
…hPeriphery

Signed-off-by: Pablo Carmona Gonzalez <pablocarmonagonzalez@gmail.com>
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.

Reset bias clamped to non-negative in PulsedRPUDeviceMetaParameter::initialize may block valid negative decay point

2 participants