Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

WIP: Meta gga #1027

Draft
wants to merge 18 commits into
base: develop
Choose a base branch
from
Draft

WIP: Meta gga #1027

wants to merge 18 commits into from

Conversation

toxa81
Copy link
Collaborator

@toxa81 toxa81 commented Nov 6, 2024

TODO list

  • derivation and documentation of MT matrix elements for vtau correction @toxa81 for LAPW
  • draft implementation of matrix elements; logically, it should be part of Hamiltonian0
  • re-use Laplacian radial function calculation (need to compute a couple of derivatives and combine them into a final radial function)
  • draft MT contribution to LAPW hamiltonain
  • interstitial contribution from vtau to LAPW Hamiltonian @toxa81
  • store tau as spin-up, -dn Periodic_function in Denisty class @abussy
  • consider moving other radial integrals from atom and atom_symmetry_class to Hamiltonian0
  • draft implementation of tau calculation
  • add contribution to tau from core electrons
  • need to compute gradient of wave-functions for core electrons; on a simple side - those are spherically symmetric states; on the difficult side - they are 2-component Dirac spinors

@toxa81
Copy link
Collaborator Author

toxa81 commented Nov 6, 2024

It's strange that there are so many commits, definitely from the unrelated changes. It will make sense to make the diff of the current changes and reapply them again on a clean fork. Also, before making more changes to the actual code, PR has to pass ci/cd

@simonpintarelli
Copy link
Collaborator

It's strange that there are so many commits, definitely from the unrelated changes. It will make sense to make the diff of the current changes and reapply them again on a clean fork. Also, before making more changes to the actual code, PR has to pass ci/cd

Seems the branch is attached to a commit 1yr ago:
2024-11-11-091943_1390x260_scrot

@abussy
Copy link
Collaborator

abussy commented Nov 11, 2024

Just so that we are on the same line @toxa81, you take care of tau functionals for FLAPW overall? We wouldn't want to duplicate work. In any case, what you do in b84eb3b is cleaner than what I was cooking up.

@toxa81
Copy link
Collaborator Author

toxa81 commented Nov 11, 2024

@abussy can you have a look how to base changes on top of clean development branch. What we have how is bad

@abussy
Copy link
Collaborator

abussy commented Nov 12, 2024

Fixed the mess with the commits. You'll have to overwrite your local branch with this one before pushing anything (otherwise we risk creating the mess again)

@toxa81
Copy link
Collaborator Author

toxa81 commented Nov 12, 2024

Fixed the mess with the commits. You'll have to overwrite your local branch with this one before pushing anything (otherwise we risk creating the mess again)

How I should do it?

@abussy
Copy link
Collaborator

abussy commented Nov 12, 2024

On your local meta-gga branch: git fetch --all, followed by git reset --hard {remote}/meta-gga, where {remote} refers to this github repo (typically origin or upstream). You can check with git remote -v

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants