Fix test-warnings related to symmetry #3355
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix a call of structure symmetrization when symmetry is not analyzed
In Attention should be paid to some certain PR during integrated testing as they may pose some warning. #3242 @Zhuxuegang2022 mentioned the warning of case
704_LJ_CR
introduced in Feature&Refactor: Structure symmetrization during relax #3247, it is because inESolver_LJ
the symmetry does not be analyzed even if symmetry=1 is set in the INPUT file.So I changed the condition to call structure symmetrization only when there exist some symmetry operations, i.e.
nrotk>0
.Change the reference value of some cases: the difference is just because the symmetrization is changed from real space into reciprocal space in Fix: symmetrize charge density in reciprocal space in any case #3081. The old values can be reproduced using real-space symmetrization in my local test.