[
https://issues.apache.org/jira/browse/MATH-1657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17708445#comment-17708445
]
François Laferrière commented on MATH-1657:
-------------------------------------------
I agree that there should not be public or protected setter. My mistake.
I also agree that my gradient example is a bit misleading. I have developed an
alternate Gradient (and Hessian) API. This is not the place to discuss it.
Forget it.
> Enhance management the objective function of optimizer
> ------------------------------------------------------
>
> Key: MATH-1657
> URL: https://issues.apache.org/jira/browse/MATH-1657
> Project: Commons Math
> Issue Type: Improvement
> Components: legacy
> Affects Versions: 4.0-beta1
> Reporter: François Laferrière
> Priority: Minor
>
> In MultivariateOptimizer class, objective function is not accessible once
> set: only computeObjectiveValue(double[] params) allows to access the value.
> This is probably to ensure that the evaluation counter is always properly
> incremented.
> But it is not very convenient for my purpose where I need to apply operators
> to the function. For instance
> {code:java}
> double[] gradient = gradientOperator.operate(getObjectiveFunction())
> {code}
> I suggest to wrap the function into an anonymous MultivariateFunction at
> setting time:
> {code:java}
> protected void setObjectiveFunction(MultivariateFunction func) {
> function = new MultivariateFunction() {
> @Override
> public double value(double[] point) throws
> IllegalArgumentException {
> incrementEvaluationCount();
> return func.value(point);
> }
> };
> }
> {code}
> So that we can still have at least a protected accessor
> {code:java}
> protected MultivariateFunction getObjectiveFunction() {
> return function;
> }
> {code}
> This is only an extension that has no impact on current API.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)