-
Notifications
You must be signed in to change notification settings - Fork 414
Update cluster gains once per cluster #1886
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…gain and mark_and_update_partial_gain routines
QoR comparison before and after gain update changes: No significant changes in pack time. |
@sfkhalid : there are warnings in the code that need to be fixed to pass CI. |
Yep, I'm looking into fixing those now. |
…ng null pointer doesn't appear
QoR Results Comparisons: |
Description
Currently during clustering the gain values are updated once for each level in a t_pb - from the root node down to the child node. However, the gain information only needs to be stored for the root pb, - the gain stored for the child pbs is never used. This pull request fixes the excess updates done during clustering.