Changeset 5d18f3 for src/Actions/PotentialAction/FitCompoundPotentialAction.def

Timestamp:

Sep 13, 2025, 2:44:01 PM (6 weeks ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.0, stable

Children:

9f1c84

Parents:

5b8c49

git-author:

Frederik Heber <frederik.heber@…> (08/30/25 15:56:12)

git-committer:

Frederik Heber <frederik.heber@…> (09/13/25 14:44:01)

Message:

FIX: PotentialTrainer did not heed take-best correctly.

• we used the unset threshold for comparison instead of the passed _threshold parameter.
• threshold needs to 0. when only best of how many/take best should be heeded.
• TEST: This fixes failing test case for fit compound potential.

File:

• src/Actions/PotentialAction/FitCompoundPotentialAction.def (modified) (1 diff)

src/Actions/PotentialAction/FitCompoundPotentialAction.def

@@ -33 +33 @@
 (STLVectorValidator< std::vector<const element *> >(1,99, ElementValidator())) \
 (PositiveValidator<unsigned int>()) \
-(PositiveValidator<double>()) \
+(PositiveValidator<double>() && RangeValidator<double>(0.,1.)) \
 (PositiveValidator<unsigned int>())