unit tests needed for RotatedSPOs
Open
testing
- Dominant language
- C++
- Stars
- 403
- Forks
- 154
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 90
Description
As #2345, RotatedSPOs is gaining functionality but we need unit tests.
I think currently the LCAO + rotation is in production and thus we need a unit test to cover this.
@spinedaflores could you make one?
test_MO.cpp was for LCAO and we can create test_RotatedSPOs_LCAO.cpp.
Evaluation functions for energy and derivatives should be included.
Contributor guide
Research direction
Start by reading test_MO.cpp and the RotatedSPOs LCAO and rotation implementation referenced in the issue. Add the proposed test_RotatedSPOs_LCAO.cpp coverage for energy and derivative evaluation, and verify that the unit tests pass for the production LCAO-plus-rotation path.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- hpc, testing-qa
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100