Skip to content

Periodic EEQ-BC implementation complete - #3

Merged
thfroitzheim merged 73 commits into
thfroitzheim:eeq-bcfrom
lmseidler:eeq-bc
Aug 18, 2025
Merged

Periodic EEQ-BC implementation complete#3
thfroitzheim merged 73 commits into
thfroitzheim:eeq-bcfrom
lmseidler:eeq-bc

Conversation

@lmseidler

Copy link
Copy Markdown
  • finished implementation of periodic functions
  • added tests to pbc implementation

@thfroitzheim thfroitzheim left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks already good. Mainly a few formatting things.

Comment thread src/multicharge/model/eeq.f90
Comment thread src/multicharge/model/eeq.f90
Comment thread src/multicharge/model/eeqbc.f90 Outdated
Comment thread src/multicharge/model/eeqbc.f90 Outdated
Comment thread src/multicharge/model/eeqbc.f90 Outdated
Comment thread test/unit/test_model.f90 Outdated
Comment thread test/unit/test_pbc.f90 Outdated
Comment thread test/unit/test_pbc.f90 Outdated
Comment thread test/unit/test_pbc.f90 Outdated
Comment thread test/unit/test_pbc.f90 Outdated
lmseidler and others added 14 commits August 15, 2025 16:41
Co-authored-by: Thomas Froitzheim <92028749+thfroitzheim@users.noreply.github.com>
Co-authored-by: Thomas Froitzheim <92028749+thfroitzheim@users.noreply.github.com>
Co-authored-by: Thomas Froitzheim <92028749+thfroitzheim@users.noreply.github.com>
Co-authored-by: Thomas Froitzheim <92028749+thfroitzheim@users.noreply.github.com>
Co-authored-by: Thomas Froitzheim <92028749+thfroitzheim@users.noreply.github.com>
Co-authored-by: Thomas Froitzheim <92028749+thfroitzheim@users.noreply.github.com>
Comment thread src/multicharge/model/eeqbc.f90 Outdated
@thfroitzheim
thfroitzheim merged commit 8a223cc into thfroitzheim:eeq-bc Aug 18, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants