Skip to content

Fix: Update function calls in pyabacus to align with new function signature in hpsi_func#5176

Merged
mohanchen merged 3 commits into
deepmodeling:developfrom
a1henu:develop
Sep 26, 2024
Merged

Fix: Update function calls in pyabacus to align with new function signature in hpsi_func#5176
mohanchen merged 3 commits into
deepmodeling:developfrom
a1henu:develop

Conversation

@a1henu

@a1henu a1henu commented Sep 26, 2024

Copy link
Copy Markdown

Reminder

  • Have you linked an issue with this pull request?
  • Have you added adequate unit tests and/or case tests for your pull request?
  • Have you noticed possible changes of behavior below or in the linked issue?

Linked Issue

Fix #5175

What's changed?

In pull request #3134, the function signature in hpsi_func has changed, leading to the pyabacus::hsolver failures. This pull request fix it

@mohanchen mohanchen added Features Needed The features are indeed needed, and developers should have sophisticated knowledge Bugs Bugs that only solvable with sufficient knowledge of DFT and removed Features Needed The features are indeed needed, and developers should have sophisticated knowledge labels Sep 26, 2024
@mohanchen mohanchen merged commit 3b2e5ad into deepmodeling:develop Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugs Bugs that only solvable with sufficient knowledge of DFT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pyabacus::hsolver failed due to changes of the order of [in\out] parameters in hpsi_func

2 participants