Skip to content

Tests: Use sum_cube.py in place of sum_cube.cpp#6948

Merged
mohanchen merged 1 commit intodeepmodeling:developfrom
Cstandardlib:tests/sum_cube
Feb 3, 2026
Merged

Tests: Use sum_cube.py in place of sum_cube.cpp#6948
mohanchen merged 1 commit intodeepmodeling:developfrom
Cstandardlib:tests/sum_cube

Conversation

@Cstandardlib
Copy link
Collaborator

@Cstandardlib Cstandardlib commented Feb 3, 2026

What's changed?

  • Replace sum_cube.cpp with sum_cube.py for integrate tests.
  • Now no need to compile sum_cube.

AsTonyshment
AsTonyshment approved these changes Feb 3, 2026
View reviewed changes
Copy link
Collaborator

@AsTonyshment AsTonyshment left a comment

Choose a reason for hiding this comment

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

LGTM. Nice job.

@mohanchen mohanchen added Refactor Refactor ABACUS codes Features Needed The features are indeed needed, and developers should have sophisticated knowledge labels Feb 3, 2026
@mohanchen mohanchen merged commit d4b15bc into deepmodeling:develop Feb 3, 2026
14 checks passed
@Cstandardlib Cstandardlib deleted the tests/sum_cube branch February 3, 2026 08:49
@Cstandardlib Cstandardlib mentioned this pull request Mar 14, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mohanchen mohanchen mohanchen approved these changes

@AsTonyshment AsTonyshment AsTonyshment approved these changes

Assignees

No one assigned

Labels

Features Needed The features are indeed needed, and developers should have sophisticated knowledge Refactor Refactor ABACUS codes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Cstandardlib @mohanchen @AsTonyshment