Skip to content
Snippets Groups Projects

Refactoring of the SIC code:

Merged Nicolas Tancogne-Dejean requested to merge reorganize_sic into main
All threads resolved!

Description

Refactoring of the SIC code:

  • The ADSIC is now properly computed for the unpolarized case and now also computes the energy.
  • The polarized and unpolarized cases are now giving exaclty the same result for close-shell systems.
  • A test is added for this.

News snippet

Bugfix for ADSIC in the unpolarized case. ADSIC now properly computes the energy.

Checklist

  • I have checked that my code follows the Octopus coding standards
  • I have added tests for all the new features added in this request.

Merge request reports

Pipeline #633469860 passed

Pipeline passed for 2535a3cd on reorganize_sic

Approved by

Merged by Martin LuedersMartin Lueders 2 years ago (Sep 7, 2022 2:02pm UTC)

Merge details

  • Changes merged into main with 3ffa4996.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #633588578 created

Pipeline created for 3ffa4996 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Martin Lueders
  • added 1 commit

    Compare with previous version

  • Martin Lueders resolved all threads

    resolved all threads

  • Martin Lueders approved this merge request

    approved this merge request

  • Martin Lueders enabled an automatic merge when the pipeline for 2535a3cd succeeds

    enabled an automatic merge when the pipeline for 2535a3cd succeeds

  • Martin Lueders mentioned in commit 3ffa4996

    mentioned in commit 3ffa4996

  • mentioned in issue #555 (closed)

  • mentioned in issue #455 (closed)

  • mentioned in issue #458 (closed)

  • Please register or sign in to reply
    Loading