WIP Bug bubble #16
Open
WIP Bug bubble #16
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- The eliashberg_product_fft now uses indices instead of interpolation. But this comes with the cost of giving a Gamma with the correct wmesh. The sum over omega for the constant part was substituted by already known Fourier transformation. Also fixed 10. - We now use semi-randomized inputs for the eigenvalue solver to get rid of the tail fit problems.
- Restructure the eliashberg module to be more flexible - Add power method - Test eliashberg_product and the eigenvalue solver individually in seperate tests. - Add documentation
The constant part was included in matrix product like summation.
Also: - use this macro in cmake files - add cmakefile for eliashberg tests - add matrix_rpa.py as a module - minor eliashberg test changes - adjust to new folder hierarchy
- If the plot is done in eliashberg/product_summation_vs_fft is now regulated by a parameter. - The solve_eliashberg method does now take initial deltas as input to start the iterative solvers. It only uses the one produced by semi_randomized_initial_delta if none is given to it.
- Add a first implementation for bandstructure like plotting WIP - Add a matplotlib style sheet to use with the notebooks - Add a user guide that uses the linearized Eliashberg equation WIP
If the dynamic part of the pairing vertex is zero the solving of the eliashberg equation will only be done with the constant part reducing the computational cost.
test: - the eigenvectors can differ in sign depending on when the eigenvaluesolver stops. Therefore check for sign flip. - missing parenthesis eliashberg.py: - stopped using variable `eliashberg_product` or multiple things
- minor change in plotting_tools.py so it does not access the [0,0] of the Gf, object
Lose the unnecessary model class and make `SquareLattice` a function called `create_square_lattice`.
- norb is now the number of orbitals EXCLUDING spin to make the function less error prone.
- high symmetry points are no longer part of the bsplot implementation and the user gives now the k-points himself. - axes tuning added - dos implementaion added
This reverts commit 88989a3.
The two-particle Green's functions do not require to have a omega and momentum mesh but can have an arbitrary one.
Instead of using loops over the indicies we are reordering indices to use matrix products. This speeds up the construction significantly. Also the singlet and triplet function now call the same creation function but with different factors for charge and spin.
This reverts commit a270b16.
-clang-format edited lines
Wentzell
pushed a commit
that referenced
this pull request
Nov 15, 2024
* [ghactions] Update runner images and compiler version
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Do not merge