Fix a bug and a magic number in module_exx_symmetry #5848
Merged
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
mohanchen
approved these changes
Jan 11, 2025
mohanchen
added
Bugs
dyzheng
pushed a commit
to dyzheng/abacus-develop
that referenced
this pull request
Jan 14, 2025
* fix a magic number in get_euler_angle * do not allow higher symmetry of bvk supercell than the original cell
dyzheng
added a commit
that referenced
this pull request
Jan 14, 2025
…x` updates in develop branch (#5859) * Fix: get_pchg_lcao with nspin=4 * Fix: out_mul with DFT+U * Enable the support of singular number of electrions in DeepKS orbital label (#5793) * delete test print (#5795) * delete test print * change the boundry condition * Refactor: update logic of init_chg (#5801) * Fix: optimize lr_spectrum (#5805) * update the broadening function in lr_spectrum * optimize transition analysis and fix norm bug * fix some warnings about esolver (#5807) * Fix the Segmentation fault caused by zero atom case under LCAO basis. (#5821) * add const in module_lr (#5817) * Fix: support negative value in parse_expression (#5826) * Use template to reconstruct parse_expression * Feature: output R matrix at each MD step * Modify'matrix_HS' to 'matrix' for R matrix output * Merge branches 'develop' and 'develop' of https://github.com/1041176461/abacus-develop into develop * Fix: modify index in parse_expression * Fix: add regfree for parse_expression * Doc: update phonopy doc * Doc: update phonopy doc * fix tdos plot for nspin=2 * optimize dosplot for nspin=2 * fix legend for dosplot * Add files via upload * Update cal_edm_tddft.cpp * Refactor: modify exx loop for relax/md * Update result.ref * Fix wrong parameters in integrate test * Update result.ref * Update result.ref * Update result.ref * Update result.ref * Update result.ref * Update result.ref * Update result.ref * Update result.ref * add exx when istep>0 * Update dos.py * Update esolver_sdft_pw.cpp * Update lcao_before_scf.cpp * Update Exx_LRI_interface.h * Update Exx_LRI_interface.hpp * Fix: compile error * Fix: compile error * Fix: change HSE relax/md result.ref for new framework * Fix: compile error * compatible with exx_iter_finish * Add files via upload * Update esolver_ks_lcao_tddft.h * Update esolver_ks_lcao_tddft.cpp * Fix: support negative value in parse_expression * [pre-commit.ci lite] apply automatic fixes --------- Co-authored-by: jiyuang <[email protected]> Co-authored-by: Qianrui <[email protected]> Co-authored-by: HTZhao <[email protected]> Co-authored-by: maki49 <[email protected]> Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com> * Fix: dmin is replaced by dmax (#5829) * fix #5819 * Fix a bug and a magic number in module_exx_symmetry (#5848) * fix a magic number in get_euler_angle * do not allow higher symmetry of bvk supercell than the original cell --------- Co-authored-by: Liu Renxi <[email protected]> Co-authored-by: liiutao <[email protected]> Co-authored-by: Yu Liu <[email protected]> Co-authored-by: LUNASEA <[email protected]> Co-authored-by: Liang Sun <[email protected]> Co-authored-by: jiyuyang <[email protected]> Co-authored-by: jiyuang <[email protected]> Co-authored-by: Qianrui <[email protected]> Co-authored-by: HTZhao <[email protected]> Co-authored-by: maki49 <[email protected]> Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com> Co-authored-by: Haozhi Han <[email protected]>
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.
This PR fixes the following cases:
exxsym-bug-case-BN-y.zip
exxsym-bug-case-2Si2-k122.zip
Thanks for the help of @1041176461 @JGHan7 @PeizeLin