Skip to content

Commit f9daa06

Browse files
committed
#1477 make fix better
1 parent 5726251 commit f9daa06

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

pybamm/solvers/base_solver.py

+6-7
Original file line numberDiff line numberDiff line change
@@ -608,10 +608,9 @@ def jacp(*args, **kwargs):
608608

609609
# if we have changed the equations to include the explicit sensitivity
610610
# equations, then we also need to update the mass matrix
611-
n_state_without_sens = model.len_rhs_and_alg
612611
if calculate_sensitivities_explicit:
613612
n_inputs = model.len_rhs_sens // model.len_rhs
614-
if model.mass_matrix.shape[0] == n_state_without_sens:
613+
if model.mass_matrix.shape[0] == model.len_rhs_and_alg:
615614
model.mass_matrix_inv = pybamm.Matrix(
616615
block_diag(
617616
[model.mass_matrix_inv.entries] * (n_inputs + 1),
@@ -625,14 +624,14 @@ def jacp(*args, **kwargs):
625624
)
626625
else:
627626
# take care if calculate_sensitivites used then not used
628-
if model.mass_matrix.shape[0] > n_state_without_sens:
627+
if model.mass_matrix.shape[0] > model.len_rhs_and_alg:
629628
model.mass_matrix_inv = pybamm.Matrix(
630-
model.mass_matrix_inv.entries[:n_state_without_sens,
631-
:n_state_without_sens]
629+
model.mass_matrix_inv.entries[:model.len_rhs_and_alg,
630+
:model.len_rhs_and_alg]
632631
)
633632
model.mass_matrix = pybamm.Matrix(
634-
model.mass_matrix.entries[:n_state_without_sens,
635-
:n_state_without_sens]
633+
model.mass_matrix.entries[:model.len_rhs_and_alg,
634+
:model.len_rhs_and_alg]
636635
)
637636

638637
# Save CasADi functions for the CasADi solver

0 commit comments

Comments
 (0)