refactor: better parameter ordering

Author: MilesCranmer

src/MLJInterface.jl

@@ -77,8 +77,8 @@ function modelexpr(
             numprocs::Union{Int,Nothing} = nothing
             procs::Union{Vector{Int},Nothing} = nothing
             addprocs_function::Union{Function,Nothing} = nothing
-            worker_timeout::Union{Real,Nothing} = nothing
             heap_size_hint_in_bytes::Union{Integer,Nothing} = nothing
+            worker_timeout::Union{Real,Nothing} = nothing
             worker_imports::Union{Vector{Symbol},Nothing} = nothing
             logger::Union{AbstractSRLogger,Nothing} = nothing
             runtests::Bool = true
@@ -300,8 +300,8 @@ function _update(
         numprocs=m.numprocs,
         procs=m.procs,
         addprocs_function=m.addprocs_function,
-        worker_timeout=m.worker_timeout,
         heap_size_hint_in_bytes=m.heap_size_hint_in_bytes,
+        worker_timeout=m.worker_timeout,
         worker_imports=m.worker_imports,
         runtests=m.runtests,
         saved_state=(old_fitresult === nothing ? nothing : old_fitresult.state),

src/SearchUtils.jl

@@ -118,8 +118,8 @@ end
     numprocs::Union{Int,Nothing}=nothing,
     procs::Union{Vector{Int},Nothing}=nothing,
     addprocs_function::Union{Function,Nothing}=nothing,
-    worker_timeout::Union{Real,Nothing}=nothing,
     heap_size_hint_in_bytes::Union{Integer,Nothing}=nothing,
+    worker_timeout::Union{Real,Nothing}=nothing,
     worker_imports::Union{Vector{Symbol},Nothing}=nothing,
     runtests::Bool=true,
     return_state::VRS=nothing,

src/SymbolicRegression.jl

@@ -406,15 +406,15 @@ which is useful for debugging and profiling.
     which is the number of processes to use, as well as the `lazy` keyword argument.
     For example, if set up on a slurm cluster, you could pass
     `addprocs_function = addprocs_slurm`, which will set up slurm processes.
-- `worker_timeout::Union{Real,Nothing}=nothing`: Timeout in seconds for worker processes
-    to establish connection with the master process. If `JULIA_WORKER_TIMEOUT` is already set,
-    that value is used. Otherwise defaults to `min(60, numprocs^2)`. When explicitly provided,
-    this temporarily overrides `JULIA_WORKER_TIMEOUT` only during worker creation.
 - `heap_size_hint_in_bytes::Union{Int,Nothing}=nothing`: On Julia 1.9+, you may set the `--heap-size-hint`
     flag on Julia processes, recommending garbage collection once a process
     is close to the recommended size. This is important for long-running distributed
     jobs where each process has an independent memory, and can help avoid
     out-of-memory errors. By default, this is set to `Sys.free_memory() / numprocs`.
+- `worker_timeout::Union{Real,Nothing}=nothing`: Timeout in seconds for worker processes
+    to establish connection with the master process. If `JULIA_WORKER_TIMEOUT` is already set,
+    that value is used. Otherwise defaults to `min(60, numprocs^2)`. When explicitly provided,
+    this temporarily overrides `JULIA_WORKER_TIMEOUT` only during worker creation.
 - `worker_imports::Union{Vector{Symbol},Nothing}=nothing`: If you want to import
     additional modules on each worker, pass them here as a vector of symbols.
     By default some of the extensions will automatically be loaded when needed.
@@ -473,8 +473,8 @@ function equation_search(
     numprocs::Union{Int,Nothing}=nothing,
     procs::Union{Vector{Int},Nothing}=nothing,
     addprocs_function::Union{Function,Nothing}=nothing,
-    worker_timeout::Union{Real,Nothing}=nothing,
     heap_size_hint_in_bytes::Union{Integer,Nothing}=nothing,
+    worker_timeout::Union{Real,Nothing}=nothing,
     worker_imports::Union{Vector{Symbol},Nothing}=nothing,
     runtests::Bool=true,
     saved_state=nothing,
@@ -525,8 +525,8 @@ function equation_search(
         numprocs=numprocs,
         procs=procs,
         addprocs_function=addprocs_function,
-        worker_timeout=worker_timeout,
         heap_size_hint_in_bytes=heap_size_hint_in_bytes,
+        worker_timeout=worker_timeout,
         worker_imports=worker_imports,
         runtests=runtests,
         saved_state=saved_state,