Skip to content

Commit 9709503

Browse files
fixed test + proper integration
1 parent 6ef6cdc commit 9709503

File tree

2 files changed

+11
-8
lines changed

2 files changed

+11
-8
lines changed

src/festim/hydrogen_transport_problem.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1380,14 +1380,17 @@ def mixed_term(u, v, n):
13801380
self.J = dolfinx.fem.form(J, entity_maps=entity_maps)
13811381

13821382
def create_solver(self):
1383-
self.solver = NewtonSolver(
1383+
self.solver = NewNewtonSolver(
13841384
self.forms,
1385-
self.J,
13861385
[subdomain.u for subdomain in self.volume_subdomains],
13871386
bcs=self.bc_forms,
1388-
max_iterations=self.settings.max_iterations,
1387+
J=self.J,
1388+
# max_iterations=self.settings.max_iterations,
13891389
petsc_options=self.petsc_options,
13901390
)
1391+
self.solver.atol = self.settings.atol
1392+
self.solver.rtol = self.settings.rtol
1393+
self.solver.max_it = self.settings.max_iterations
13911394

13921395
def create_flux_values_fenics(self):
13931396
"""For each particle flux create the ``value_fenics`` attribute"""
@@ -1447,7 +1450,7 @@ def iterate(self):
14471450
self.update_time_dependent_values()
14481451

14491452
# solve main problem
1450-
self.solver.solve(self.settings.atol, self.settings.rtol)
1453+
self.solver.solve()
14511454

14521455
# post processing
14531456
self.post_processing()
@@ -1475,7 +1478,7 @@ def run(self):
14751478
self.progress_bar.refresh() # refresh progress bar to show 100%
14761479
else:
14771480
# Solve steady-state
1478-
self.solver.solve(self.settings.rtol)
1481+
self.solver.solve()
14791482
self.post_processing()
14801483

14811484
def __del__(self):

test/system_tests/test_multi_material.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ def c_exact_bot_np(x):
142142

143143
my_model.temperature = 500.0 # lambda x: 300 + 10 * x[1] + 100 * x[0]
144144

145-
my_model.settings = F.Settings(atol=None, rtol=1e-5, transient=False)
145+
my_model.settings = F.Settings(atol=1e-5, rtol=1e-5, transient=False)
146146
my_model.exports = [
147147
F.VTXSpeciesExport(f"u_{subdomain.id}.bp", field=H, subdomain=subdomain)
148148
for subdomain in my_model.volume_subdomains
@@ -216,7 +216,7 @@ def test_1_material_discontinuous_version():
216216

217217
my_model.temperature = lambda x: 300 + 100 * x[0]
218218

219-
my_model.settings = F.Settings(atol=None, rtol=1e-5, transient=False)
219+
my_model.settings = F.Settings(atol=1e-5, rtol=1e-5, transient=False)
220220

221221
my_model.exports = [
222222
F.VTXSpeciesExport(
@@ -384,7 +384,7 @@ def test_2_mats_particle_flux_bc():
384384

385385
my_model.temperature = lambda x: 300 + 10 * x[1] + 100 * x[0]
386386

387-
my_model.settings = F.Settings(atol=None, rtol=1e-5, transient=False)
387+
my_model.settings = F.Settings(atol=1e-5, rtol=1e-5, transient=False)
388388

389389
my_model.exports = [
390390
F.VTXSpeciesExport(f"u_{subdomain.id}.bp", field=H, subdomain=subdomain)

0 commit comments

Comments
 (0)