Skip to content
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

refactor!: Remove mean reduction from MultiEigenStepperLoop #3671

Merged

Conversation

andiwand
Copy link
Contributor

@andiwand andiwand commented Oct 1, 2024

After #3521 we also want to change the default and remove the weighted mean reduction as it is unstable.

This pull request primarily focuses on removing the WeightedComponentReducerLoop struct and its associated functionality from the MultiEigenStepperLoop class and its tests. The changes simplify the codebase by eliminating the weighted component reduction logic and replacing it with the MaxWeightReducerLoop.

Removal of WeightedComponentReducerLoop:

Replacement with MaxWeightReducerLoop:

Test updates:

@github-actions github-actions bot added the Component - Documentation Affects the documentation label Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

📊: Physics performance monitoring for 6c14959

Full contents

physmon summary

Copy link
Member

@benjaminhuth benjaminhuth left a comment

Choose a reason for hiding this comment

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

Thanks for having this in mind!

@andiwand andiwand mentioned this pull request Oct 1, 2024
Copy link

sonarcloud bot commented Oct 1, 2024

@kodiakhq kodiakhq bot merged commit dd33d4c into acts-project:main Oct 1, 2024
42 checks passed
@github-actions github-actions bot removed the automerge label Oct 1, 2024
@andiwand andiwand deleted the remove-multieigenstepper-mean-reduction branch October 1, 2024 12:25
@paulgessinger paulgessinger modified the milestones: next, v37.0.0 Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Core Affects the Core module Component - Documentation Affects the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants