Skip to content

bypass amplification.bin load in pla when uniform factor is provided#1756

Merged
sstruzik merged 2 commits intomainfrom
fix/pla_uniform
Aug 27, 2025
Merged

bypass amplification.bin load in pla when uniform factor is provided#1756
sstruzik merged 2 commits intomainfrom
fix/pla_uniform

Conversation

@sstruzik
Copy link
Copy Markdown
Contributor

@sstruzik sstruzik commented Aug 22, 2025

bypass amplification.bin load in post loss amplification when uniform factor is provided

When a uniform factor is provided, amplification.bin load is not needed. This PR remove this step and allow user to provide a uniform factor without the need to create the amplification.bin file

@sstruzik sstruzik self-assigned this Aug 22, 2025
@sstruzik sstruzik added bug Something isn't working LTS - 2.3 labels Aug 22, 2025
@sstruzik sstruzik moved this to Waiting for Review in Oasis Dev Team Tasks Aug 22, 2025
@sstruzik sstruzik merged commit d9db46f into main Aug 27, 2025
33 of 34 checks passed
@github-project-automation github-project-automation bot moved this from Waiting for Review to Done in Oasis Dev Team Tasks Aug 27, 2025
@sambles sambles mentioned this pull request Aug 29, 2025
@awsbuild awsbuild added this to the 2.4.7 milestone Aug 29, 2025
sambles pushed a commit that referenced this pull request Nov 7, 2025
…1756)

* bypass amplification.bin load in pla when uniform factor is provided
@sambles sambles deleted the fix/pla_uniform branch January 29, 2026 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working LTS - 2.3 LTS - 2.4

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

PLA amplification file should not be required when uniform factor is given

4 participants