[
https://issues.apache.org/jira/browse/MATH-1657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17707927#comment-17707927
]
Gilles Sadowski commented on MATH-1657:
---------------------------------------
{quote}The getObjectiveFunction() does not circumvent the counting
{quote}
OK, my mistake. Sorry.
Yet I don't understand the (protected) "setter". IIUC, you only need the
"getter" (?).
{quote}No, the current API is not covering the case.
{quote}
Unless I'm mistaken (again), your example shows a way to compute de gradient
that circumvents the current API, where the gradient operator is passed with a
dedicated type of {{{}OptimizationData{}}}.
> 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)
