Skip to content

Fix SCAN-L Laplacian: port to new module_xc structure + rebase#7652

Closed
mintleaf84 wants to merge 1 commit into
deepmodeling:developfrom
mintleaf84:fix-scanl-laplacian-rebase
Closed

Fix SCAN-L Laplacian: port to new module_xc structure + rebase#7652
mintleaf84 wants to merge 1 commit into
deepmodeling:developfrom
mintleaf84:fix-scanl-laplacian-rebase

Conversation

@mintleaf84

Copy link
Copy Markdown

Port PR #7457 (SCAN-L laplacian) to latest develop branch with manual rebase.

Changes:

  • Add laplacian_rho(), FD Laplacian kernel stress/potential correction
  • Register SCANL in xc_functional.cpp, detect XC_FLAGS_NEEDS_LAPLACIAN
  • tau_xc/tau_xc_spin: pass lapl/vlapl parameters
  • v_xc_meta: return 5-tuple with voflapl
  • Unit test test_xc6.cpp for SCANL sensitivity and libxc reference values
  • Extract FD kernel into shared compute_fd_gg()

Tested: 6/8 XC tests pass (VXC needs MPI runtime)

…te SCANL test refs

- Extract duplicated FD Laplacian kernel computation into
  XC_Functional::compute_fd_gg() (xc_functional.h/cpp)
- Replace raw new[]/delete[] with std::vector in laplacian_rho()
- Fix test_xc6.cpp reference values to match current libxc version
- All XC tests pass (6/8, VXC needs MPI runtime)
Copilot AI review requested due to automatic review settings July 17, 2026 14:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mintleaf84

Copy link
Copy Markdown
Author

Superseded by updating PR #7457 with rebased changes

@mintleaf84 mintleaf84 closed this Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants