Skip to content
Snippets Groups Projects

1D singularity

Merged Nicolas Tancogne-Dejean requested to merge singularity_1D into develop
All threads resolved!

Description

Adding the support of the treatment of the Coulomb singularity in 1D. This helps the convergence with respect to k-points of Hartree-Fock calculations.

News snippet

Adding the support of the treatment of the Coulomb singularity in 1D.

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 #492399855 passed

Pipeline passed for 0a9a15e1 on singularity_1D

Merged by Sebastian OhlmannSebastian Ohlmann 3 years ago (Mar 15, 2022 10:50am UTC)

Loading

Pipeline #492531839 passed

Pipeline passed for 15093061 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Codecov Report

    Merging #1645 (0a9a15e1) into develop (171ac317) will increase coverage by 0.00%. The diff coverage is 92.15%.

    Impacted file tree graph

    @@           Coverage Diff            @@
    ##           develop    #1645   +/-   ##
    ========================================
      Coverage    66.79%   66.80%           
    ========================================
      Files          539      539           
      Lines        96522    96549   +27     
    ========================================
    + Hits         64471    64495   +24     
    - Misses       32051    32054    +3     
    Impacted Files Coverage Δ
    src/hamiltonian/singularity.F90 82.11% <92.00%> (+1.70%) :arrow_up:
    src/poisson/poisson_fft.F90 68.47% <100.00%> (+0.08%) :arrow_up:

    Continue to review full report at Codecov.

    Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 171ac31...0a9a15e. Read the comment docs.

    Edited by Codecov
  • added 1 commit

    • d388b457 - Adding a test to cover the new code.

    Compare with previous version

  • Sebastian Ohlmann resolved all threads

    resolved all threads

  • Sebastian Ohlmann approved this merge request

    approved this merge request

  • Sebastian Ohlmann enabled an automatic merge when the pipeline for d388b457 succeeds

    enabled an automatic merge when the pipeline for d388b457 succeeds

  • Nicolas Tancogne-Dejean aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • added 1 commit

    • 0a9a15e1 - Adding a test to cover the new code.

    Compare with previous version

  • @sohlmann I had to fix a warning. Do you mind approving again this merge request?

  • Sebastian Ohlmann approved this merge request

    approved this merge request

  • mentioned in commit 15093061

  • Please register or sign in to reply
    Loading