Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix: Fix the Segmentation fault caused by zero atom case with LCAO basis. #5821

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

sunliang98
Copy link
Collaborator

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?
  • Have you explained the changes of codes in core modules of ESolver, HSolver, ElecState, Hamilt, Operator or Psi? (ignore if not applicable)

Linked Issue

Fix #5711

Unit Tests and/or Case Tests for my changes

  • An integrate test 223_NO_zero_atom has been added.

What's changed?

  • Example: My changes might affect the performance of the application under certain conditions, and I have tested the impact on various scenarios...

Any changes of core modules? (ignore if not applicable)

  • Example: I have added a new virtual function in the esolver base class in order to ...

@mohanchen mohanchen added Bugs Bugs that only solvable with sufficient knowledge of DFT Refactor Refactor ABACUS codes Input&Output Suitable for coders without knowing too many DFT details labels Jan 6, 2025
@mohanchen mohanchen merged commit 9732d79 into deepmodeling:develop Jan 6, 2025
14 checks passed
@sunliang98 sunliang98 deleted the zero_atom branch January 6, 2025 08:14
kluophysics pushed a commit to kluophysics/abacus-develop that referenced this pull request Jan 10, 2025
dyzheng pushed a commit to dyzheng/abacus-develop that referenced this pull request Jan 14, 2025
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
Labels
Bugs Bugs that only solvable with sufficient knowledge of DFT Input&Output Suitable for coders without knowing too many DFT details Refactor Refactor ABACUS codes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

not output force and stress with kpacing
2 participants