Skip to content

Commit

Permalink
Refactor as suggested
Browse files Browse the repository at this point in the history
Co-authored-by: Brady Planden <[email protected]>
  • Loading branch information
NicolaCourtier and BradyPlanden authored Jul 3, 2024
1 parent a02257a commit 6e9ad43
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions pybop/problems/fitting_problem.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,15 +92,16 @@ def evaluate(self, inputs: Inputs):
inputs = self.parameters.verify(inputs)

requires_rebuild = False
for key in inputs.keys():
if (
key in self._model.rebuild_parameters
and inputs[key] != self.parameters[key].value
):
self.parameters[key].update(value=inputs[key])
requires_rebuild = True
for key, value in inputs.items():
if key in self._model.rebuild_parameters:
current_value = self.parameters[key].value
if value != current_value:
self.parameters[key].update(value=value)
requires_rebuild = True

if requires_rebuild:
self._model.rebuild(parameters=self.parameters)
self._model.rebuild(parameters=self.parameters)

y = self._model.simulate(inputs=inputs, t_eval=self._time_data)

Expand Down

0 comments on commit 6e9ad43

Please sign in to comment.