Skip to content

Commit

Permalink
remove public set_parameters
Browse files Browse the repository at this point in the history
  • Loading branch information
MarcBerliner committed Dec 2, 2024
1 parent 63d138d commit 66766b8
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/pybamm/simulation.py
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ def build(self, initial_soc=None, inputs=None):
self._model_with_set_params = self._model
self._built_model = self._model
else:
self.set_parameters()
self._set_parameters()
self._mesh = pybamm.Mesh(self._geometry, self._submesh_types, self._var_pts)
self._disc = pybamm.Discretisation(
self._mesh, self._spatial_methods, **self._discretisation_kwargs
Expand Down
8 changes: 4 additions & 4 deletions tests/unit/test_simulation.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ def test_basic_ops(self):
assert V.has_symbol_of_classes(pybamm.Parameter)
assert not V.has_symbol_of_classes(pybamm.Matrix)

sim.set_parameters()
sim._set_parameters()
assert sim._mesh is None
assert sim._disc is None
V = sim.model_with_set_params.variables["Voltage [V]"]
Expand Down Expand Up @@ -138,8 +138,8 @@ def test_solve_already_partially_processed_model(self):
def test_reuse_commands(self):
sim = pybamm.Simulation(pybamm.lithium_ion.SPM())

sim.set_parameters()
sim.set_parameters()
sim._set_parameters()
sim._set_parameters()

sim.build()
sim.build()
Expand All @@ -149,7 +149,7 @@ def test_reuse_commands(self):

sim.build()
sim.solve([0, 600])
sim.set_parameters()
sim._set_parameters()

def test_set_crate(self):
model = pybamm.lithium_ion.SPM()
Expand Down

0 comments on commit 66766b8

Please sign in to comment.