tree 60379b114e7d3007f6198dcf32cc15b7396574e3
parent 5cb5b35a930c1702278083c75769dbb4e5801045
author Sameer Agarwal <sameeragarwal@google.com> 1596215152 -0700
committer Sameer Agarwal <sameeragarwal@google.com> 1596215152 -0700

Fix a bug in DynamicAutoDiffCostFunction

DynamicAutoDiffCostFunction::Evaluate when provided with a jacobians
array that was non-empty but all its entries are nullptr, would
compute num_active_parameters = 0, and then skip over all the loops
that evaluated the CostFunctor.

The fix is to check if num_active_parameters == 0, and then treat
it as the case where jacobians array is null.

Thanks to Ky Waegel for reporting and providing a reproduction for this.

Change-Id: Ib86930c2c3f722724d249f662bf88238679bbf98