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.

Sign up for GitHub

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

Jump to bottom

AuxiliaryRealBlockPC and AuxiliaryComplexBlockPC #181

Merged
merged 10 commits into from
Mar 27, 2024
Merged

AuxiliaryRealBlockPC and AuxiliaryComplexBlockPC #181

merged 10 commits into from
Mar 27, 2024

Conversation

JHopeCollins
Copy link
Collaborator

@JHopeCollins JHopeCollins commented Mar 21, 2024 •
edited
Loading

Pulling out the auxiliary operator PC for a real-valued block from #152 and implementing it with the auxiliary operator for the complex blocks already in asQ.

Todo:

  • asQ.preconditioners file for auxiliary block pcs
  • test for complex block pc
  • test for real block pc
  • tidy up example script for aux pcs (remove commented code)
  • update other scripts/tests/pcs with new pc names and appctx keys

@JHopeCollins JHopeCollins added enhancement New feature or request Core functionality Adding to the main paradiag functionality labels Mar 21, 2024
@JHopeCollins JHopeCollins self-assigned this Mar 21, 2024
@JHopeCollins JHopeCollins mentioned this pull request Mar 21, 2024
Closed
4 tasks
@JHopeCollins JHopeCollins merged commit 894b19a into master Mar 27, 2024
1 check passed
@JHopeCollins JHopeCollins deleted the auxiliaryblockpc branch March 27, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@colinjcotter colinjcotter colinjcotter approved these changes

Assignees

@JHopeCollins JHopeCollins

Labels
Core functionality Adding to the main paradiag functionality enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JHopeCollins @colinjcotter