Rename sweep_solve -> sweep_solve! to obey convention

Author: Jack Dunham

src/solvers/applyexp.jl

@@ -91,7 +91,7 @@ function applyexp(
   ]
   sweep_iter = SweepIterator(init_prob, kws_array)
 
-  converged_prob = sweep_solve(sweep_callback, sweep_iter)
+  converged_prob = sweep_solve!(sweep_callback, sweep_iter)
 
   return state(converged_prob)
 end

src/solvers/eigsolve.jl

@@ -64,7 +64,7 @@ function eigsolve(operator, init_state; nsweeps, nsites=1, outputlevel=0, sweep_
     state=align_indices(init_state), operator=ProjTTN(align_indices(operator))
   )
   sweep_iter = SweepIterator(init_prob, nsweeps; nsites, outputlevel, sweep_kwargs...)
-  prob = sweep_solve(sweep_iter)
+  prob = sweep_solve!(sweep_iter)
   return eigenvalue(prob), state(prob)
 end
 

src/solvers/fitting.jl

@@ -90,7 +90,7 @@ function fit_tensornetwork(
   kwargs_array = [(; sweep_kwargs..., extra_sweep_kwargs..., sweep) for sweep in 1:nsweeps]
 
   sweep_iter = SweepIterator(init_prob, kwargs_array)
-  converged_prob = sweep_solve(sweep_iter)
+  converged_prob = sweep_solve!(sweep_iter)
 
   return rename_vertices(inv_vertex_map(overlap_network), ket(converged_prob))
 end

src/solvers/sweep_solve.jl

@@ -4,7 +4,7 @@ default_sweep_callback(sweep_iterator) = sweep_iterator
 
 # In this implementation the function `sweep_solve` is essentially just a wrapper around 
 # the iterate interface that allows one to pass callbacks.
-function sweep_solve(
+function sweep_solve!(
   sweep_iterator;
   sweep_callback=default_sweep_callback,
   region_callback=default_region_callback,
@@ -21,18 +21,18 @@ end
 
 # I suspect that `sweep_callback` is the more commonly used callback, so allow this to
 # be set using the `do` syntax.
-function sweep_solve(
+function sweep_solve!(
   sweep_callback, sweep_iterator; region_callback=default_region_callback
 )
-  return sweep_solve(sweep_iterator; sweep_callback, region_callback)
+  return sweep_solve!(sweep_iterator; sweep_callback, region_callback)
 end
 
-function sweep_solve(
+function sweep_solve!(
   each_region_iterator::EachRegion; region_callback=default_region_callback
 )
-  return sweep_solve(region_callback, each_region_iterator)
+  return sweep_solve!(region_callback, each_region_iterator)
 end
-function sweep_solve(region_callback, each_region_iterator::EachRegion)
+function sweep_solve!(region_callback, each_region_iterator::EachRegion)
   for _ in each_region_iterator
     # I don't think it is obvious what object this particular callback should take,
     # but for now be consistant and pass the parent sweep iterator.