Skip to content

Conversation

@Cstandardlib
Copy link
Collaborator

@Cstandardlib Cstandardlib commented Oct 10, 2025

Linked Issue

Fix #6578

Important information

Please extend the namespace container::lapackConnector module for Column-Major interfaces,
and the class LapackConnector for Row-Major ones.

Changes

  1. Merge class LapackWrapper in source/source_base/module_external/lapack_wrapper.hinto namespace container::lapackConnector in source/source_base/module_container/base/third_party/lapack.h.
  2. class LapackConnector in source/source_base/module_external/lapack_connector.h remains alive for Row-major interfaces.
  3. Update source/source_hsolver/kernels/hegvd_op.h to migrate from the lapack_wrapper to the lapackConnector.

@Cstandardlib Cstandardlib marked this pull request as ready for review October 10, 2025 11:36
@Cstandardlib Cstandardlib added Useful Information Useful information for others to learn/study Refactor Refactor ABACUS codes Interfaces Interfaces with other packages labels Oct 10, 2025
@mohanchen mohanchen merged commit 265dac7 into deepmodeling:develop Oct 10, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Interfaces Interfaces with other packages Refactor Refactor ABACUS codes Useful Information Useful information for others to learn/study

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reduce duplicate LAPACK Interfaces

2 participants